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.

Sign up for GitHub

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

Jump to bottom

Address -Wall warnings #679

Merged
merged 28 commits into from
Dec 26, 2019
Merged

Address -Wall warnings #679

merged 28 commits into from
Dec 26, 2019

Conversation

wxmerkt
Copy link
Collaborator

@wxmerkt wxmerkt commented Dec 25, 2019

Activates -Wall for dependent projects and addresses all warnings.

@wxmerkt
[exotica_collision_scene_fcl] Address sign comparison
4ca93c3
@wxmerkt
[exotica_aico_solver] Address sign comparison, unused return, correct…
45190e6 
… brackets
@wxmerkt
[exotica_ddp_solver] Address sign comparison, unused variables
a3e0a3d
@wxmerkt
[exotica_ompl_solver] Address sign comparison
423a83e
@wxmerkt
[exotica_time_indexed_rrt_connect] Address sign comparison, unused va…
30c7aa4 
…riables
@wxmerkt
[exotica_examples] Address unused variables
717d85c
@wxmerkt
[exotica_core] KinematicElement: Initialisation order
d30e289
@wxmerkt
[exotica_python] Add get_mass
965b2b2
@wxmerkt
[exotica_python] Update for latest pybind11_catkin
47e9df6
@wxmerkt
[exotica_core] Export Wall to dependant projects
44b4f24
@wxmerkt
[exotica_core_task_maps] Address Wall warnings
9a49f14
@wxmerkt
[exotica_core] Do not warn for maybe-unitialized
94116fc
@wxmerkt
[exotica] Update authors in metapackage
96da752
@wxmerkt
[exotica_ddp_solver] Fix strict aliasing warning by directly using Te…
43aaf00 
…nsorMap
@wxmerkt
[exotica_cartpole_dynamics_solver] Address unused variables
e8cd845
@wxmerkt
[exotica_pendulum_dynamics_solver] Address unused variables
6991889
@wxmerkt
[exotica_pinocchio_dynamics_solver] Address unused variables
527b88a
@wxmerkt
[exotica_quadrotor_dynamics_solver] Address unused variables, re-use …
c4babdf 
…common computation
@wxmerkt
[exotica_collision_scene_fcl_latest] Address unused variables and sig…
a95e94f 
…n comparison
@wxmerkt
[exotica_ilqr_solver] Address unused variables
e496cca
@wxmerkt
[exotica_levenberg_marquardt_solver] Address sign comparison
1d6d68f
@wxmerkt
[exotica_ompl_control_solver] Address sign comparison and unused vari…
074fa5c 
…ables
@wxmerkt
[exotica_dynamics_solver] Add metapackage for dynamics solvers
0987e50
@wxmerkt
[exotica_python] Buildfix for 2.3.1 and 2.4.3 releases
b92b462
@wxmerkt
[exotica_dynamics_solvers] Add double integrator
456a712
@wxmerkt
[exotica_examples] Update required dependencies
4e8d2a6
@wxmerkt
Update maintainers
778d774
@wxmerkt
[exotica_examples] Fix python_ik_talos.launch permissions
ad73269
@wxmerkt wxmerkt merged commit 020b16f into master Dec 26, 2019
@wxmerkt wxmerkt deleted the wxm-address-Wall-warnings branch December 26, 2019 16:09
wxmerkt added a commit that referenced this pull request Jun 22, 2020
@wxmerkt
Merge pull request #679 from ipab-slmc/wxm-address-Wall-warnings
2225d75 
Address -Wall warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@wxmerkt