Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- 09:07 < gus_> do I fill a bugreport?
- 09:08 < Daniel_S> gus_: the scanner will go around initializing all your classes, they need to be able to survive that without preconditions (e.g. windows being shown)
- 09:10 < gus_> it would appear it doesn't, in the gdb I can see screen is 0x0 (NULL)
- 09:11 < gus_> it dereferences it and gets killed
- 09:13 < Daniel_S> gus_: well, I can't promise it isn't a gtk bug, but what calls GDK_SCREEN_GET_CLASS (screen)->get_setting ?
- 09:13 < Daniel_S> the fact that there is not screen doesn't seem unexpected
- 09:14 < gus_> this is the bt (gtk and evince with debug symbols, glib20 wout them) http://pastebin.com/yVDiQTUN
- 09:17 < Daniel_S> gus_: EvViewPresentation needs to delay whatever it's doing in ev_view_presentation_class_init (or just hack around the issue by not doing things when there's no screen)
- 09:18 < gus_> do you think it is then an evince bug or simply evince triggered a bug in gdk? I ask because I have a bugrep ready to be fired
- 09:20 < Daniel_S> gus_: when we encountered similar issues in GEGL we concluded that it was our bug and made our singletons lazy to account for it
- 09:20 < Daniel_S> gus_: I can't speak for what the introspection maintainers think though :)
- 09:20 < gus_> not sure if introspection or evince
- 09:21 < Daniel_S> gus_: the issue is that introspection needs to be able to initialize all your GTypes but will not call any initialization functions (gtk or your own) before doing that
- 09:23 < Daniel_S> gus_: so hypothetically it's a introspection bug because they could provide some way to initialize things, but the current design just says you can't require
- pre-initialization for classes you want to introspect
- 09:24 < Daniel_S> it was not a huge deal for us to sanitize things, if it's more painful for you I don't see a reason not to harass the introspection guys a bit first :)
- 09:28 < gus_> I'm doing that right now :)
- 09:28 < gus_> thanks for your input Daniel_S
Advertisement
Add Comment
Please, Sign In to add comment