After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 700444 - [W32] The use of uninitialized GDK default screen
[W32] The use of uninitialized GDK default screen
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Backend: Win32
unspecified
Other Windows
: Normal normal
: ---
Assigned To: gtk-win32 maintainers
gtk-bugs
: 701121 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-05-16 10:58 UTC by LRN
Modified: 2013-06-11 09:16 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix display/screen/displaymanager init interdependency (903 bytes, patch)
2013-05-16 10:58 UTC, LRN 		committed Details | Review
Pass screen to visual init (no default screen is available yet) (2.11 KB, patch)
2013-05-16 10:59 UTC, LRN 		committed Details | Review

Description LRN 2013-05-16 10:58:07 UTC 
Some functions in GDK rely on having default display, default screen, etc.
These default objects are initialized by _gdk_win32_display_open(). However, _gdk_win32_display_open() sets them at the end:
g_signal_emit_by_name (_gdk_display, "opened");
, and they can't be used until that moment. Yet sub-initialization routines _gdk_win32_display_open() uses depend on these default objects.
Comment 1 LRN 2013-05-16 10:58:38 UTC 
Created attachment 244387 [details] [review]
Fix display/screen/displaymanager init interdependency
Comment 2 LRN 2013-05-16 10:59:03 UTC 
Created attachment 244388 [details] [review]
Pass screen to visual init (no default screen is available yet)
Comment 3 Matthias Clasen 2013-06-07 11:18:41 UTC 
Review of attachment 244387 [details] [review]:

makes sense to me
Comment 4 Matthias Clasen 2013-06-07 11:19:01 UTC 
Review of attachment 244388 [details] [review]:

makes sense
Comment 5 Matthias Clasen 2013-06-07 11:21:58 UTC 
*** Bug 701121 has been marked as a duplicate of this bug. ***
Comment 6 Alexander Larsson 2013-06-11 09:16:33 UTC 
Attachment 244387 [details] pushed as ae35951 - Fix display/screen/displaymanager init interdependency
Attachment 244388 [details] pushed as b001625 - Pass screen to visual init (no default screen is available yet)