Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compile warnings #4

Merged
merged 1 commit into from
Aug 26, 2015
Merged

Fix compile warnings #4

merged 1 commit into from
Aug 26, 2015

Conversation

syohex
Copy link

@syohex syohex commented Aug 26, 2015

I got following compile warnings.

/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp: In member function ‘int GetFile::ConnectHTTP()’:
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:64:6: warning: unused variable ‘r’ [-Wunused-variable]
  int r;
      ^
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp: In member function ‘int GetFile::GetHTTPFile(const string&)’:
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:170:6: warning: unused variable ‘r_head’ [-Wunused-variable]
  int r_head = 1;
      ^
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:164:41: warning: ignoring return value of ‘ssize_t write(int, const void*, size_t)’, declared with attribute warn_unused_result [-Wunused-result]
  write(sock, send_buf, strlen(send_buf));
                                         ^
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:166:41: warning: ignoring return value of ‘ssize_t write(int, const void*, size_t)’, declared with attribute warn_unused_result [-Wunused-result]
  write(sock, send_buf, strlen(send_buf));
                                         ^
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:168:41: warning: ignoring return value of ‘ssize_t write(int, const void*, size_t)’, declared with attribute warn_unused_result [-Wunused-result]
  write(sock, send_buf, strlen(send_buf));
                                         ^
/home/syohei/work/Autoware/ros/src/data/packages/map_db/lib/map_db/GetFile.cpp:192:2: warning: ‘i’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  for(int i; i < 10; i++) {
  ^

...

/home/syohei/work/Autoware/ros/src/computing/planning/motion/packages/driving_planner/nodes/lattice_trajectory_gen/lattice_trajectory_gen.cpp:402:29: warning: ‘geometry_msgs::Twist calcTwist(int)’ defined but not used [-Wunused-function\
]
 static geometry_msgs::Twist calcTwist(int next_waypoint)
                             ^
Linking CXX executable /home/syohei/work/Autoware/ros/devel/lib/driving_planner/lattice_trajectory_gen
[ 98%] Built target lattice_trajectory_gen
Scanning dependencies of target lattice_twist_convert
[ 98%] Building CXX object computing/planning/motion/packages/driving_planner/CMakeFiles/lattice_twist_convert.dir/nodes/lattice_twist_convert/lattice_twist_convert.cpp.o
/home/syohei/work/Autoware/ros/src/computing/planning/motion/packages/driving_planner/nodes/lattice_twist_convert/lattice_twist_convert.cpp: In function ‘void splineCallback(const ConstPtr&)’:
/home/syohei/work/Autoware/ros/src/computing/planning/motion/packages/driving_planner/nodes/lattice_twist_convert/lattice_twist_convert.cpp:212:12: warning: unused variable ‘start_time’ [-Wunused-variable]
     double start_time = ros::Time::now().toSec();
            ^

syohex added a commit that referenced this pull request Aug 26, 2015
@syohex syohex merged commit d88f6c0 into master Aug 26, 2015
@syohex syohex deleted the fix-compile-warnings branch August 26, 2015 09:28
aohsato added a commit that referenced this pull request Feb 6, 2019
…rom 776a358..5b5a876

5b5a876 Update README.md
a311caf [Fix] Remapping on VLP-32C driver (#6)
43318fd merge upstream (#7)
a23e1cb Updated README with TravisCI correct image
036f15f unpack fix
8204e75 README conflict
d610f80 Merge branch 'master' into Autoware
016eac4 Merge branch 'master' of https://github.com/ros-drivers/velodyne
7bfc2fe Revert "Update from remote (#3)" (#4)
8d6788a Update from remote (#3)
cce2135 1.5.0
72fc452 Updating CHANGELOGs for new release.
9bcd061 Merge pull request #164 from ros-drivers/maint/vlp_32c_support
d53984a Merge pull request #187 from moooeeeep/master
4dd510e Merge pull request #189 from kveretennicov/patch-1
80bb8aa Fix malformed plugin description XML
15771a4 bugfix: getPacket() function is expected to return negative value on error
e71a38d 1.4.0
c4d0a94 Updating changelogs for new release.
bfb035a Merge branch 'master' into maint/vlp_32c_support
797eead Merge pull request #178 from sts-thm/bugfix_issue_#174
6fead99 Merge branch 'master' into maint/vlp_32c_support
0296313 Merge pull request #177 from C-NR/feature/WrapPointcloudData
ab52485 Removed debug outputs
de15234 Changes fixing deadlock for specific cut_angle values.
a67d5cf moved definition of VPoint and VPointCloud back to namespace rawdata in rawdata.h
de79cb0 put a wrapper around pointcloud data including a generic setter method to enable the use of arbitrary data structures  (pcl pointcloud, depth image, octomaps and so on) to be filled by just using RawData::unpack method with the wrapper object as parameter
8c34bb7 Merge branch 'master-origin' into velodyne-master
8feed85 Merge pull request #135 from cfneuhaus/bugfix
ef2345e Last fix for README. For now.
9d02a43 Fixing markdown in README.md.
75de270 Changing README to markdown.
9515c24 Adding Travis badge to README.
7c3095e Merge pull request #172 from ros-drivers/maint/add_melodic_build
291f33e Removing outdated build_script.sh.
0417778 Updating README.
1ba6fde Adding Melodic to the list of Travis builds.
850948d Merge pull request #170 from ros-drivers/maint/move_header_files
7cc9a9b Moving header files to traditional location inside include folders.
2463922 Adding distance_resolution to test yaml files.
37cc837 Adding VLP-32C support.
9a444bd Merge pull request #161 from ros-drivers/maint/updating_travis_to_industrial_ci
7494535 Learned that ROS_REPO is preferred. Added Lunar build.
2dab9a1 Merge branch 'master' into maint/updating_travis_to_industrial_ci
f06ca46 Merge pull request #160 from ros-drivers/maint/updating_package_xml_to_v2
dd56d4c Updating .travis.yml after testing. Lowering 64e Hz test threshold.
4c48049 First test of industrial_ci.
8a6d1c7 Updated all package.xmls to ver 2. Cleaned up catkin_lint errors.
e04afa4 Merge pull request #151 from Axel13fr/feature/No_Communication_Diag_Update
f0df814 Merge branch 'awesomebytes-fix_32c_packet_rate'
a1d6f18 Update and rename README.rst to README.md
52c0a0d README format
1a70413 Merge branch 'master' into Autoware
7976d12 support vlp32c now
6b4242b Fix packet rate for the Velodyne 32C
277035c Conventions: adding name for unused method parameter.
118c9a2 Added a periodic update of the diagnostics so that when no data is received at all from the Velodyne, a diagnostic information will be published. The previous implementation would publish diagnostics only on packet reception.
273520e Added hdl32c, fixed naming
e21b522 Merge pull request #146 from stsundermann/patch-2
0e5a200 Merge pull request #150 from ros-drivers/mikaelarguedas-patch-1
db6b5ee update to use non deprecated pluginlib macro
560fe12 Use std::abs instead of fabsf
559d0eb bugfix: when no device ip is set, we still want to filter by udp port.

git-subtree-dir: ros/src/sensing/drivers/lidar/packages/velodyne
git-subtree-split: 5b5a8765ba826e2fd33b65f7381eef670a7f940e
gbiggs pushed a commit that referenced this pull request Jun 20, 2019
@mitsudome-r mitsudome-r added the version:autoware-ai Autoware.AI label Jun 14, 2022
kenji-miyake referenced this pull request in tier4/autoware Aug 10, 2022
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
nabetetsu pushed a commit to nabetetsu/autoware that referenced this pull request Aug 23, 2022
taikitanaka3 pushed a commit to taikitanaka3/autoware that referenced this pull request Jun 14, 2023
…dation#4)

* chore(repos): autoware.repos for autoware_common

* ci(pre-commit-ansible): upgrade ansible-lint

* ci(pre-commit-ansible): fix errors (autowarefoundation#3490)

* add additional_dependencies

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fix ansible-lint errors

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ci(pre-commit-ansible): autoupdate (autowarefoundation#3500)

* ci(pre-commit-ansible): autoupdate

Signed-off-by: GitHub <noreply@github.com>

* apply var-naming[no-role-prefix]

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ignore colcon directories

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fixup

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Co-authored-by: awf-autoware-bot[bot] <94889083+awf-autoware-bot[bot]@users.noreply.github.com>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>
taikitanaka3 pushed a commit to taikitanaka3/autoware that referenced this pull request Jun 14, 2023
…dation#4)

* chore(repos): autoware.repos for autoware_common

* ci(pre-commit-ansible): upgrade ansible-lint

* ci(pre-commit-ansible): fix errors (autowarefoundation#3490)

* add additional_dependencies

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fix ansible-lint errors

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ci(pre-commit-ansible): autoupdate (autowarefoundation#3500)

* ci(pre-commit-ansible): autoupdate

Signed-off-by: GitHub <noreply@github.com>

* apply var-naming[no-role-prefix]

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ignore colcon directories

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fixup

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Co-authored-by: awf-autoware-bot[bot] <94889083+awf-autoware-bot[bot]@users.noreply.github.com>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>
shmpwk pushed a commit that referenced this pull request Jun 14, 2023
* fix: autoware.repos for autoware common

Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>

* chore(repos): update autoware.repos for autoware_common (#4)

* chore(repos): autoware.repos for autoware_common

* ci(pre-commit-ansible): upgrade ansible-lint

* ci(pre-commit-ansible): fix errors (#3490)

* add additional_dependencies

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fix ansible-lint errors

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ci(pre-commit-ansible): autoupdate (#3500)

* ci(pre-commit-ansible): autoupdate

Signed-off-by: GitHub <noreply@github.com>

* apply var-naming[no-role-prefix]

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ignore colcon directories

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fixup

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>

---------

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Co-authored-by: awf-autoware-bot[bot] <94889083+awf-autoware-bot[bot]@users.noreply.github.com>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>

---------

Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: Yu Asabe <yuasabe@g.ecc.u-tokyo.ac.jp>
Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Co-authored-by: awf-autoware-bot[bot] <94889083+awf-autoware-bot[bot]@users.noreply.github.com>
Co-authored-by: kenji-miyake <kenji-miyake@users.noreply.github.com>
Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>
youtalk added a commit that referenced this pull request May 20, 2024
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
BalajiEdutech pushed a commit to BalajiEdutech/autoware that referenced this pull request Dec 10, 2024
BalajiEdutech pushed a commit to BalajiEdutech/autoware that referenced this pull request Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants