[Top][All Lists]

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

bug#31615: 27.0; doc of `uncomment-region(-default)'

From: Eli Zaretskii
Subject: bug#31615: 27.0; doc of `uncomment-region(-default)'
Date: Sat, 02 Jun 2018 13:44:59 +0300

> Date: Sun, 27 May 2018 21:20:04 -0700 (PDT)
> From: Drew Adams <address@hidden>
> 1. These two functions should not have the same doc string.  Function
> `*-default' should say what is - the default value of
> `uncomment-region-function'.


> 2. The doc strings should be much more specific about the behavior, in
> particular wrt ARG.  Apparently the functions do nothing at all if ARG
> is larger than the first sequence of comment chars in the region.
> E.g. for a comment such as this one:
>   ;; foo bar
> (uncomment-region (line-beginning-position) (line-end-position) 3)
> does nothing.  That's not obvious from the doc.

I'm not sure this was the intent of the code.  I'd expect it to remove
the maximum number of comment delimiters it can, not just punt

> 3. The doc string talks about "comment markers".  What's that?

It means "comment delimiters"; fixed.

> 4. Enhancement request: Provide a way to remove ALL comment chars and
> all whitespace between them, up to the first non-comment char.

Not done.

reply via email to

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