#1 Fixes for Windows build

open
nobody
None
5
2007-09-07
2007-09-07
No

David,

I am attaching a patch file against the *.tar distribution that includes the following:

* Fixes makefile.win to successfully compile gtkrxdlg.dll on Windows.

* Adds a *.def file which is needed on Windows to export the functions. Otherwise rxapi can not load the functions. (There are other ways of doing it, but this is the simplest and least prone to error.)

* Adds the option to the make file to build a debug version of the DLL.

* Fixes a number of places in the C code where uninitialized pointers were being passed into the GTK libraries causing memory exceptions.

* Fixes a spot in the *.cls file where ~new was left off. (Unfortunately there were also a few places in the *.cls file that had trailing white space. My editor strips that out. So there are few more differences in the file than technically needed. But, hey, it cleans up the file a little. <grin>)

* Adds a windows-building.txt file that documents pretty well how to build the DLL on Windows.

With the changes you can build either a release or debug DLL on Windows that works. All the testXX.rex files run without errors. I didn't do any additional testing, but running your test programs gives a good demonstration.

Discussion

  • Mark Miesfeld

    Mark Miesfeld - 2007-09-07

    Context sensitive patch against the *.tar distribution

     
  • Mark Miesfeld

    Mark Miesfeld - 2007-09-07

    Changed files for this patch plus compile DLL

     
  • Mark Miesfeld

    Mark Miesfeld - 2007-09-07

    Logged In: YES
    user_id=191588
    Originator: YES

    David,

    I am also attaching a zip file that contains just the changed files, but the complete file rather than a diff.

    It also has a release build of gtkRxDlg.dll in it. Tested on both ooRexx 3.1.2 and ooRexx 3.2.0 with your existing test programs. They all run without detectable error.
    File Added: gtkrxdlg_win_fixes_files.zip

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks