tech@mandoc.bsd.lv
 help / color / mirror / Atom feed
* Re: mdocml: Store page titles in the correct case, and by default, only put
       [not found] ` <4ED0D36B.2080503@bsd.lv>
@ 2011-11-26 12:11   ` Ingo Schwarze
  0 siblings, 0 replies; only message in thread
From: Ingo Schwarze @ 2011-11-26 12:11 UTC (permalink / raw)
  To: Kristaps Dzonsons; +Cc: tech

Hi Kristaps,

Kristaps Dzonsons wrote on Sat, Nov 26, 2011 at 12:54:19PM +0100:
> On 26/11/2011 12:23, schwarze@mdocml.bsd.lv wrote:

>> Log Message:
>> -----------
>> Store page titles in the correct case, and by default, only
>> put stuff into the database that man(1) will be able to retrieve.
>> However, support an option to use all directories and files.
>> feedback and ok  kristaps@

> Some post-checkin comments.  First, I think we can make verbose and
> use_all into global statics, as they don't change and we're just
> cluttering the function prototypes with them.  No?

Yes.
I think a small number of global static variables is fine
in a small utility program, even though lots of global statics
tend to cause confusion in big systems.
So i like your suggestion.

> And second, in the manual, can you put the default usage following
> the arguments?
> 
>  Its arguments are as follows:
> 
>  -a    blah blah
> 
>  -b    blah blah
> 
>  By default, foo does this and that...
> 
> This is much easier to read and doesn't require that operators read
> each option to understand the defaults.

Yes, that makes sense.  By the same argument, i'd even move the
defaults *before* the options (leaving the fatal errors where
they are, of course).  When you are satisfied with the defaults,
you don't even need to read the options, and unless you understand
the defaults, the options (in particular -a) won't make much sense
to you.

I'm planning to handle these two tasks after putting the
unformatted parser in and adding the type field to the index
file.

Yours,
  Ingo
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-11-26 12:11 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <201111261123.pAQBNvRl005038@krisdoz.my.domain>
     [not found] ` <4ED0D36B.2080503@bsd.lv>
2011-11-26 12:11   ` mdocml: Store page titles in the correct case, and by default, only put Ingo Schwarze

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).