[pexports-0.47] suggested PATCH (leakage, usage screen)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[pexports-0.47] suggested PATCH (leakage, usage screen)

Jannick

Attached a patch for the package pexports (version 0.47) for your consideration. This is the result of the work when trying to use and understand it.

 

·       reduced memory leakage (by using the reactivated functions ‘str_tree_free’ and ‘str_list_free’ (where the latter now returning the pointer to the copied string) in the lexer and bison parser; I am not sure why the memory housekeeping routines were dropped from an earlier version)

·       slightly enhanced usage screen to better show options

·       some minor edits

 

There is no change to the functionality of the package. I moved the definition of the ‘struct str_list’ to ‘str_tree.h’ to make it available in other files more easily. As a suggestion, all of the ‘str_list_...’ stuff could be jointly moved into one single module to avoid extern calling statements. Certainly, I do leave this with you, Keith, how you prefer to organize the package.

 

I hope that the attached patch won’t be ignored by the list server. If so I will just send an email with the content of the patch file afterwards.

 

Regards,

J.


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
MinGW-users mailing list
[hidden email]

This list observes the Etiquette found at
http://www.mingw.org/Mailing_Lists.
We ask that you be polite and do the same.  Disregard for the list etiquette may cause your account to be moderated.

_______________________________________________
You may change your MinGW Account Options or unsubscribe at:
https://lists.sourceforge.net/lists/listinfo/mingw-users
Also: mailto:[hidden email]?subject=unsubscribe

pexports-0.47.patch (4K) Download Attachment
Loading...