Results 1 to 4 of 4

Thread: Change for CL_GUIManager

  1. #1

    Default Change for CL_GUIManager

    When you create a CL_GUIManager but forget to call set_window_manager or set_theme on it before using it, it crashes. Patch attached in order to document this behavior.

    If we can do runtime checks for this mistake, that would be great, but I think it would need to happen in so many places that it's not worth it.

    A few other notes:
    - the INSTALL.win32 file is unneeded and outdated now so can be removed - the information now resides here.
    - in the configure wizard two of the options aren't clear as to their purpose:
    * "Include Unicode configurations" (can't use unicode without this?)
    * "Include static libraries using DLL runtime configurations" (no idea what this is for)
    * so, it would be nice if you add explanatory tooltips to these options
    Attached Files Attached Files

  2. #2

    Default

    Just realized that the default constructor can at least initialize the wm.
    So, new better patch attached.
    Attached Files Attached Files

  3. #3
    ClanLib Developer
    Join Date
    May 2007
    Posts
    1,824

    Default

    patch applied to clanlib 2.2 svn

    thanks

  4. #4

    Default

    Thanks! Could you get rid of INSTALL.win32 as well? As I said it's outdated (and confusing to have it there).

Similar Threads

  1. Drawing differences when using CL_GUIManager*
    By genail in forum Official ClanLib SDK Forums
    Replies: 6
    Last Post: 12-12-2009, 07:37 AM
  2. CL_Sprite change to ClanLib 0.8 and 0.9
    By rombust in forum Official ClanLib SDK Forums
    Replies: 0
    Last Post: 04-21-2009, 09:55 AM
  3. How do I change controls?
    By in forum Dink Smallwood HD
    Replies: 8
    Last Post: 08-13-2006, 11:24 PM
  4. Question about what is a change
    By in forum Other RTsoft Games
    Replies: 1
    Last Post: 10-15-2002, 11:24 PM
  5. no screen change for mowing
    By in forum Other RTsoft Games
    Replies: 0
    Last Post: 09-10-2002, 07:01 PM

Bookmarks

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •