|
|
Am 02.02.2014 20:28, schrieb Le_Forgeron:
> Dear Cousin Ricky,
>
> Your happiness is easy to satisfy, with a bit of early advertisement:
>
> --- a/vfe/unix/unixconsole.cpp
> +++ b/vfe/unix/unixconsole.cpp
> @@ -474,9 +474,11 @@ int main (int argc, char **argv)
> opts.AddCommand (*argv);
> }
>
> + PrintStatus (session);
> // set all options and start rendering
> if (session->SetOptions(opts) != vfeNoError)
> {
> + PrintStatus (session);
> fprintf(stderr,"\nProblem with option setting\n");
> for(int loony=0;loony<argc_copy;loony++)
> {
>
>
>
> The first PrintStatus dumps the classical page(s) about povray (from
> version to Support libraries). (it's a bit earlier than currently, but I
> do not find a reason to delay it to the parsing of the scene).
>
> The second is used to dump the parsing error collected during the
> examination of the command line options (and maybe more), in case of error.
>
> Windows vfe (and mac too), might enjoy such changes too.
That last sentence shows that we're doing something fundamentally wrong
here: Ideally there should be only one piece of code to fix, for all
POV-Ray front-ends.
This is probably also the reason for the confusion around FS#42: The
command-line error reporting had already been made elaborate enough, but
only in the Windows version.
Post a reply to this message
|
|