Re: [Seqan-dev] small patch for CommandLineParser
Konstantin,
thank you for your patch!
However, I believe that applying the patch to the SeqAn code base leads
to more work that is probably not justified right now. We might
incorporate the behaviour from your patch in a future (revamped) version
of the command line parser. Let me explain:
Currently, all programs expect the old behaviour and print a banner and
the short help on their own after the error message. This makes the
error message disappear before the large short help.
We plan to rewrite the command line parser in the near future and thus I
do not want to adjust all applications to this patch and then shortly
afterwards to the new command line parser interface. I will forward your
proposal to the people responsible for the new command line parser and
ask them to consider this behaviour in the new code.
Again, thanks for the effort of creating and sending in the patch, we
appreciate it and hope to incorporate your and other user's further
improvements into the library.
Bests,
Manuel
On 01/12/2012 03:00 PM, Konstantin Okonechnikov wrote:
Hi!
The patch file is attached.
The patch feature: show an error message when the required number of
command line parameters doesn't match the number of given parameters.
Best regards,
Konstantin