GNU bug report logs - #50839
28.0.50; Odd things in comp-ctxt doc string

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 50839 in the body.
You can then email your comments to 50839 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: bug-gnu-emacs <at> gnu.org
Cc: Andrea Corallo <akrl <at> sdf.org>
Subject: 28.0.50; Odd things in comp-ctxt doc string
Date: Mon, 27 Sep 2021 08:58:30 +0200

I saw that compiling with native-comp warned about a too-long doc
string, but looking at comp-ctxt-byte-func-to-func-h, it's not too long:

  (byte-func-to-func-h (make-hash-table :test #'equal) :type hash-table
                     :documentation "byte-function -> comp-func.
Needed to replace immediate byte-compiled lambdas with the compiled reference.")

But looking at the actual help buffer, I get the stuff below.

It looks like bits of the doc strings have been replaced with...  odd
things?  "lambdas" is now "lamcomp-func.dbdas".  And also the
"byte-funCL-X.ction" thing.  I haven't debugged any further; perhaps
it's obvious to someone what's going on.  (I've added Andrea to the CCs.)

comp-ctxt-byte-func-to-func-h is a native compiled Lisp function in
‘comp.el’.

(comp-ctxt-byte-func-to-func-h CL-X)

Access slot "byte-func-to-func-h" of ‘comp-ctxt’ struct

byte-funCL-X.ction ->

Needed to replace immediate byte-compiled lamcomp-func.bdas with the compiled reference.

  This function has a compiler macro ‘comp-ctxt-byte-func-to-func-h--cmacro’.
  This function does not change global state, including the match data.



In GNU Emacs 28.0.50 (build 23, x86_64-pc-linux-gnu, GTK+ Version 3.24.24, cairo version 1.16.0)
 of 2021-09-26 built on elva
Repository revision: 43ae8c828d853382bbc2a27b9e14b9fff6ba18b6
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Debian GNU/Linux 11 (bullseye)

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Message #8 received at 50839 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: 50839 <at> debbugs.gnu.org
Cc: Stefan Kangas <stefan <at> marxist.se>, Andrea Corallo <akrl <at> sdf.org>
Subject: Re: bug#50839: 28.0.50; Odd things in comp-ctxt doc string
Date: Mon, 27 Sep 2021 09:00:37 +0200

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> But looking at the actual help buffer, I get the stuff below.
>
> It looks like bits of the doc strings have been replaced with...  odd
> things?  "lambdas" is now "lamcomp-func.dbdas".  And also the
> "byte-funCL-X.ction" thing.  I haven't debugged any further; perhaps
> it's obvious to someone what's going on.  (I've added Andrea to the CCs.)

Oh, and this started happening after Stefan K's long-doc-string warning
enablement in lambda, so perhaps it's related to that (or perhaps that
just makes the problem more obvious).  (So I've added him to the CCs, too.)

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Message #11 received at 50839 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Lars Ingebrigtsen <larsi <at> gnus.org>, 50839 <at> debbugs.gnu.org
Cc: Andrea Corallo <akrl <at> sdf.org>
Subject: Re: bug#50839: 28.0.50; Odd things in comp-ctxt doc string
Date: Mon, 27 Sep 2021 02:51:09 -0700

tags 50839 fixed
close 50839 28.1
thanks

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Lars Ingebrigtsen <larsi <at> gnus.org> writes:
>
>> But looking at the actual help buffer, I get the stuff below.
>>
>> It looks like bits of the doc strings have been replaced with...  odd
>> things?  "lambdas" is now "lamcomp-func.dbdas".  And also the
>> "byte-funCL-X.ction" thing.  I haven't debugged any further; perhaps
>> it's obvious to someone what's going on.  (I've added Andrea to the CCs.)
>
> Oh, and this started happening after Stefan K's long-doc-string warning
> enablement in lambda, so perhaps it's related to that (or perhaps that
> just makes the problem more obvious).  (So I've added him to the CCs, too.)

Thanks, this is now fixed on master in commit 3d2d7e8ea2.

This was caused by trying to fill an entire paragraph as if it were a
single line.  Here, I really wish we could use `cl-assert' in subr.el,
to do something like this in `internal--format-docstring-line':

    (cl-assert (not (string-match "\n" string)))

Do we have anything like that, or do we just do something like this
instead?

    (and (string-match "\n" string)
         (error "Contains newline: %s" string))

Maybe this is okay as this should pretty much only be called from
macros?

Message #18 received at 50839 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 50839 <at> debbugs.gnu.org, Andrea Corallo <akrl <at> sdf.org>
Subject: Re: bug#50839: 28.0.50; Odd things in comp-ctxt doc string
Date: Tue, 28 Sep 2021 07:21:43 +0200

Stefan Kangas <stefan <at> marxist.se> writes:

> Do we have anything like that, or do we just do something like this
> instead?
>
>     (and (string-match "\n" string)
>          (error "Contains newline: %s" string))
>
> Maybe this is okay as this should pretty much only be called from
> macros?

I think that should be OK.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Message #21 received at 50839 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 50839 <at> debbugs.gnu.org, Andrea Corallo <akrl <at> sdf.org>
Subject: Re: bug#50839: 28.0.50; Odd things in comp-ctxt doc string
Date: Tue, 28 Sep 2021 05:28:31 -0700

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

>>     (and (string-match "\n" string)
>>          (error "Contains newline: %s" string))
>>
>> Maybe this is okay as this should pretty much only be called from
>> macros?
>
> I think that should be OK.

Thanks, I've now made that change on master.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.