Subject: Re: [Boost-bugs] [Boost C++ Libraries] #3703: Request for ability to specify description column width
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2009-12-03 10:42:38
#3703: Request for ability to specify description column width
-------------------------------+--------------------------------------------
Reporter: Chard | Owner: s_ochsenknecht
Type: Feature Requests | Status: closed
Milestone: Boost 1.42.0 | Component: program_options
Version: Boost 1.41.0 | Severity: Cosmetic
Resolution: fixed | Keywords: description column formatting
-------------------------------+--------------------------------------------
Comment(by s_ochsenknecht):
Replying to [comment:9 Chard <boost@â¦>]:
> Hi Sascha
>
> I've had a bit more of a look at the changes, and there's a couple of
points to note:
>
> 1. I had called the original description length variable:
{{{min_description_length}}}, this has been renamed to
{{{description_length}}}. The "{{{min_}}}" part does add some semantic
meaning to the usage because the length that is specified is not an
absolute.
Yes, you're right. I agree that {{{min_description_length}}} might be the
better name here. I wasn't 100% aware that the description could be
longer. I'll change that.
> 2. By tying the default argument for the description length to
{{{(default_line_length / 2)}}} it does mean that existing code that
specifies the {{{line_length}}} argument will still compile, but with a
potentially different display. The minimum description width will still
be 40, regardless of the specified {{{line_length}}}. However, I suspect
this is a minor point because for most cases the output should look
''better''.
Yes, I wouldn't change it now since it can not be detected within the
constructor if the default parameter was used or not. Or we end up with 1
or more constructor overloads, which I would like to avoid. Most likely
the user will specify all parameters or none, I guess.
Thanks & regards,
Sascha
-- Ticket URL: <https://svn.boost.org/trac/boost/ticket/3703#comment:10> Boost C++ Libraries <http://www.boost.org/> Boost provides free peer-reviewed portable C++ source libraries.
This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:02 UTC