patchBabyChess - Patches: patch #6717, Revamped babychess to work on...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #6717: Revamped babychess to work on modern GNU/Linux systems

Submitter:  Nicola Fontana <ntd>
Submitted:  Wed 07 Jan 2009 05:15:02 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open

Wed 07 Jan 2009 05:15:02 PM UTC, original submission:  

The provided patch lets BabyChess be compiled on modern system (I'm using gcc-4.1.2 with gtk+-2.14.5 on a 64bit platform). I tried to separate the job in smaller patches but the work to do was not worth the effort.

A lot of things do not work but at least I can play against the BabyChess engine and (more important for me) I can analyze PGN saved games. The following are the logical enhancement done:

  1. changed casts from pointer to int to cast from int to pointer (casting pointer to integer is not valid on 64bit platform)
  2. removed hardcoded flags from dep.make
  3. dropped gnomeui dependency and used the new gtk+ equivalent: by using gtk_show_about_dialog(), gtk+-2.6 or better is a requirement.
  4. avoided direct inclusion of gtkitemfactory.h (doesn't work anymore): would be better to always include <gtk/gtk.h> only but, albeit only gtkitemfactory was generating errors
  5. updated the signal system using the GObject API instead of the deprecated Gtk+ one (anyway this was a non-blocking problem)
  6. got rid of custom inttypes and moved the _STD defines directly in dep.make as -D options for gcc: the old custom inttypes.hh did not work, probably because the system inttypes.h was yet included without _STD defines
  7. rearranged headers to let the stream_file.hh template system work: I've not deeph knowledge of C++ internals, so maybe this could also be resolved in a more elegant way.
Nicola Fontana <ntd>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #17220:  2009.patch added by ntd (84KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ntd (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-01-07 ntd Attached File- Added 2009.patch, #17220

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code