[ scummvm-Patches-2802670 ] GSOC09 extraction tool patch

SourceForge.net noreply at sourceforge.net
Mon Jun 8 01:38:54 CEST 2009


Patches item #2802670, was opened at 2009-06-07 23:38
Message generated for change (Tracker Item Submitted) made by remere
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2802670&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: GSoC
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Remere (remere)
Assigned to: Remere (remere)
Summary: GSOC09 extraction tool patch

Initial Comment:
Little late due to misunderstanding on my part.

Just a patch of changing command line arguments for one tool (I've already converted almost all of them, keeping it brief for clarity), also contains the util file, which all the tools use for their CLI parsing (and path handling). Most time was spent thinking and fighting my fingers from breaking the coding conventions. ;)

It is questionable if the util.h/.cpp should contain C++ code, I originally wrote it to use only C. But then I realized all tools use C++ anyways (infact, forcing C compiler almost causes all tools to fail), and chose to create a Filename class for clarity. Kept the STL or any advanced features out of it, so even the weakest C++ compiler should be able to compile it without problems (I think).

Oh, and I already have SVN access, submitting it here to hear your opinions.




----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2802670&group_id=37116




More information about the Scummvm-tracker mailing list