|
From: <br...@ph...> - 2006-08-13 05:11:32
|
Petr Mikulik wrote:
> Few minor things:
>
> 1. In ./configure, shouldn't strings "(EXPERIMENTAL)" be removed now? Won't
> be "nice" for 4.2.
It would be even less nice to lie about the maturity of those options.
Yes, those features are experimental, and they should be marked as such.
> 2. Gnuplot supports both exists('a') and exist('a'), but documents only one
> of them. As it was me who made this confusion, I would propose to keep only
> exist('a').
This would have been a much better idea a couple of weeks ago than it is
now. I'm afraid I have to object to changing the syntax this shortly
before the release.
> 3. doc2tex.c produces the following in the .tex file:
>
> (see {\bf start-up (p.~\pageref{start-up})\index{up}}) and, of course,
> by later explicit changes.
> see {\bf mouse variables (p.~\pageref{mouse variables})\index{variables}}
> for details.
> See {\bf fit error\_estimates (p.~\pageref{fit error_estimates})\index{estimates}}.
>
> as you see, there appears "up" keyword in the index, and similar. See the
> code in doc2tex.c, below line:
> /* Make the final word an index entry also */
> I propose to keep the whole word including "-" and "_".
OK for '-', but '_' may need closer inspections. People have abused '_'
to tweak multi-word index entries and help nodes in somewhat random ways
for a while now. I really don't think we need stuff like plot_datafile
as an index entry.
|