[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Nano-devel] [PATCH 3/3] docs: an attempt at updating the documentat

From: Benno Schulenberg
Subject: Re: [Nano-devel] [PATCH 3/3] docs: an attempt at updating the documentation for the changed defaults
Date: Thu, 17 Jan 2019 20:02:22 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1

Op 17-01-19 om 09:47 schreef David Ramsey:
> These look good for the most part, although I haven't had time to test
> them yet.  However:
> * In patch 0002, there are several traces of the
> --disable-wrapping-as-root option that aren't removed when they should
> be: the ENABLE_NANORC && DISABLE_ROOTWRAPPING #ifdef in nano,c:main(),
> and the DISABLE_ROOTWRAPPING #ifdef in rcfile.c:do_rcfiles().

Thanks.  The first one (in nano.c) was removed by patch 0001, but I will
move it to patch 0002, to have all the rootwrapping removal in one patch.
The second one (in rcfile.c) I indeed overlooked.

> * In patch 0003,there's a typo:
>     +## To always have a blank like below the title bar:


But, starting to change the documentation, I realized that things become
quite confusing when some options are set by default while all others
are not.  The options in a nanorc file are still manageable, but how do
we say in the 'nano --help' output that --nonewlines and --morespace and
--smooth are in fact no-ops because they are the default?  What would be
needed here is to have the opposite options.  So I am thinking of adding:

-e, --emptyline (below title bar),
-f, --finalnewline,
-j, --jumpyscrolling.

Their counterpart options would continue to be recognized but would be
deprecated (in the man pages and other docs) and would not be listed in
the help text.

Any opinions on that?

The --nowrap option doesn't need a counterpart because it already has
one: --fill=<number>.  (Although that options is really two things in
one: it sets the fill width for justifying, and switches on automatic
hard-wrapping.)  But if later such a counterpart would be needed:

-b, --breaklonglines.


Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]