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

usage of multiple static variables is discouraged #33

Open
rhaschke opened this issue Apr 3, 2016 · 0 comments
Open

usage of multiple static variables is discouraged #33

rhaschke opened this issue Apr 3, 2016 · 0 comments

Comments

@rhaschke
Copy link

rhaschke commented Apr 3, 2016

I'm double-posting this issue, which shows up in pluginlib usage: ros/pluginlib#37

Using static pluginlib ClassLoader defers lib unloading to program shutdown time.
But as the order of static releases is undefined, the program might crash.

IMHO, class_loader shouldn't use a handful of unrelated static vars, but collect them within a singleton class, which is only freed when all loaded libs were unloaded.
This was referenced Apr 3, 2016
Merged
Closed
rhaschke added a commit to ubi-agni/robot_model that referenced this issue Apr 7, 2016
@rhaschke
don't use a static ClassLoader instance
68c7615 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
guihomework pushed a commit to ubi-agni/robot_model that referenced this issue Aug 25, 2016
@rhaschke @guihomework
don't use a static ClassLoader instance
defb812 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
rhaschke added a commit to ubi-agni/robot_model that referenced this issue Sep 13, 2016
@rhaschke
don't use a static ClassLoader instance
debcbcb 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
@jonbinney jonbinney mentioned this issue Oct 17, 2016
Merged
rhaschke added a commit to ubi-agni/urdf that referenced this issue Dec 19, 2017
@rhaschke
don't use a static ClassLoader instance
0822499 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
rhaschke added a commit to ubi-agni/urdf that referenced this issue Aug 10, 2018
@rhaschke
don't use a static ClassLoader instance
c987046 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
@Rauldg Rauldg mentioned this issue Oct 31, 2018
Closed
rhaschke added a commit to ubi-agni/urdf that referenced this issue Feb 8, 2019
@rhaschke
don't use a static ClassLoader instance
0c74998 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
rhaschke added a commit to ubi-agni/urdf that referenced this issue Oct 7, 2020
@rhaschke
don't use a static ClassLoader instance
a56b286 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
rhaschke added a commit to ubi-agni/urdf that referenced this issue Sep 21, 2022
@rhaschke
don't use a static ClassLoader instance
4aa551e 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
rhaschke added a commit to ubi-agni/urdf that referenced this issue Sep 22, 2022
@rhaschke
don't use a static ClassLoader instance
e022abc 
- crashes on exit: ros/class_loader#33
- on-demand-unloading works with ros/class_loader#34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rhaschke