emacs-bug-tracker
[Top][All Lists]
Advanced

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

[debbugs-tracker] bug#28781: closed (25.3; Override erroneous "incorrect


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#28781: closed (25.3; Override erroneous "incorrect verb voice" from checkdoc)
Date: Wed, 11 Oct 2017 13:11:02 +0000

Your message dated Wed, 11 Oct 2017 16:09:37 +0300
with message-id <address@hidden>
and subject line Re: bug#28781: 25.3; Override erroneous "incorrect verb voice" 
from checkdoc
has caused the debbugs.gnu.org bug report #28781,
regarding 25.3; Override erroneous "incorrect verb voice" from checkdoc
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
28781: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=28781
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 25.3; Override erroneous "incorrect verb voice" from checkdoc Date: Tue, 10 Oct 2017 10:52:18 -0700
I have the following definition:

    (defmacro straight--functionp (object)
      "Non-nil if OBJECT, an unquoted symbol, evaluates to a function.
    However, if OBJECT evaluates to its own symbol value or t, then
    return nil. This is useful for allowing a function to be called
    with nil, non-nil, or a function object, without worrying about
    the non-nil value being interpreted as a function: just call the
    function with the quoted name of the argument, or use t."
      (let ((object-sym (make-symbol "object")))
        `(let ((,object-sym ,object))
           (and (not (memq ,object-sym '(,object t)))
                (functionp ,object-sym)))))

M-x checkdoc reports the following on the first line:

    Probably "evaluates" should be imperative "evaluate"

This is wrong. How can I suppress this spurious warning, other than by
rewording the docstring? I know that checkdoc has
`checkdoc-symbol-words' for suppressing spurious warnings about unquoted
symbols, but there appears to be nothing similar I can customize in
file-local variables for this case.



--- End Message ---
--- Begin Message --- Subject: Re: bug#28781: 25.3; Override erroneous "incorrect verb voice" from checkdoc Date: Wed, 11 Oct 2017 16:09:37 +0300
> From: Radon Rosborough <address@hidden>
> Date: Tue, 10 Oct 2017 13:31:33 -0700
> Cc: address@hidden
> 
> 
> [1:text/plain Hide]
> 
> > > If so, maybe the docstring for `functionp' should be changed (after
> > > all, it was what I modeled this one off of):
> > >
> > >     Non-nil if OBJECT is a function.
> >
> > Yes, I think so.
> 
> The patch is attached.

Thanks, pushed.


--- End Message ---

reply via email to

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