search

darrenlafreniere / lafarren-image-completer

Implementation of the research titled: "Image Completion Using Efficient Belief Propagation via Priority Scheduling and Dynamic Pruning"
http://www.lafarren.com/image-completer/
GNU General Public License v3.0
32 stars 18 forks source link

Consider using Boost Program Options to parse arguments #18

Closed daviddoria closed 13 years ago

daviddoria commented 13 years ago

It seems like you have reinvented the wheel a little bit :) Using standard libraries like this makes the code easier to extend.

http://www.boost.org/doc/libs/1_46_0/doc/html/program_options.html

I have some examples here: http://programmingexamples.net/index.php?title=CPP/Boost/ProgramOptions/Simple

darrenlafreniere commented 13 years ago

That's why I originally used wxCmdLineParser :). I cringe at bringing in another giant dependency when there's already one giant dependency with wxWidgets. I'd rather go back to using wxCmdLineParser and use wxStrings at that level only, and convert to std::string after it's parsed.

darrenlafreniere commented 13 years ago

Which should be pretty easy now that the Unicode mysteries are solved.

daviddoria commented 13 years ago

Sounds good to me. I was actually wondering where that stuff went haha. Sure, there is no reason that someone should know Boost Program Options any more than they would know wxCmdLineParser, but at least using one of them gives SOMEONE a chance to know it. No one knows Lafarren-tech :)

darrenlafreniere commented 13 years ago

A full wxWidgets -> Boost conversion is a possibility in the future, since Boost is more widely used.

darrenlafreniere commented 13 years ago

lol, Lafarren-tech forever!!

daviddoria commented 13 years ago

Sounds good. Boost is a nightmare to learn, but it is nice once you know it.

daviddoria commented 13 years ago

Not necessary.