Hi! >Was this on purpose?
Nope - didn't see it until later. I pretty much re-formatted everything (spaces->tabs), so they will probably be difficult to merge. Is the change big? Regards, Klaus Post http://www.klauspost.com On Sun, Aug 12, 2012 at 3:19 AM, Camilo Polymeris <[email protected]>wrote: > On Wed, Jul 18, 2012 at 11:34 AM, Camilo Polymeris <[email protected]> > wrote: > > On Wed, Jul 18, 2012 at 10:20 AM, Klaus Post <[email protected]> > wrote: > >> Hi! > >> > >> > >> It looks really great, and implemented really cleanly! > >> > >> I agree output settings would be nice. IMO it could be added later, as a > >> modified version on the "Export As..." dialog, where the directory and > >> filename options are removed. > > > > Well, this "client mode" was mostly meant to be called from another > > program, and be as transparent as possible. Maybe it would make sense > > to be able to pass scaling or max size on the commandline, too. > > > >> The only small thing I noticed was that other commandline options are > >> removed (or not valid as far as I can tell) - not a serious limitation, > but > >> that should probably be adjusted. > > > > Which other commandline options? Apart from those I implemented, I > > only see the usual GTK commandline options, like --class, --name, etc, > > and they seem to work for me. > > > >> Otherwise I'm all for including it. > > I just noticed you applied the patch to the svn repo. But did so with > my first patch, which does not include output format options and such, > like the second one. See: > http://bugzilla.rawstudio.org/show_bug.cgi?id=556 > > Was this on purpose? > > Regards, > Camilo > > _______________________________________________ > Rawstudio-dev mailing list > [email protected] > http://rawstudio.org/cgi-bin/mailman/listinfo/rawstudio-dev >
_______________________________________________ Rawstudio-dev mailing list [email protected] http://rawstudio.org/cgi-bin/mailman/listinfo/rawstudio-dev
