Re: [Seqan-dev] small patch for CommandLineParser

[Manuel Holtgrewe <manuel.holtgrewe@fu-berlin.de>]

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