GNU bug report logs - #42777
28.0.50; Obsolete eldoc-message function used in CEDET

Previous Next

Package: emacs;

Reported by: Lars Ingebrigtsen <larsi <at> gnus.org>

Date: Sun, 9 Aug 2020 15:24:02 UTC

Severity: normal

Found in version 28.0.50

Fixed in version 28.1

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

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 09 Aug 2020 15:24:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Lars Ingebrigtsen <larsi <at> gnus.org>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Sun, 09 Aug 2020 15:24:02 GMT) Full text and rfc822 format available.

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
Subject: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 09 Aug 2020 17:23:01 +0200
In semantic-idle-summary-idle-function:
cedet/semantic/idle.el:771:10: Warning: ‘eldoc-message’ is an obsolete
    function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.

In semantic-idle-summary-refresh-echo-area:
cedet/semantic/idle.el:802:13: Warning: ‘eldoc-message’ is an obsolete
    function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.

I guess this is the result of João's changes in July?  I had a quick
peek at how to fix it, but it wasn't immediately obvious to me.


In GNU Emacs 28.0.50 (build 51, x86_64-pc-linux-gnu, GTK+ Version 3.24.20, cairo version 1.16.0)
 of 2020-08-09 built on xo
Repository revision: 1a845a672dc73c8e98e6cb9bb734616e168e60ba
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12008000
System Description: Debian GNU/Linux bullseye/sid


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





Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Fri, 28 Aug 2020 17:56:02 GMT) Full text and rfc822 format available.

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

From: Stefan Kangas <stefan <at> marxist.se>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org,
 João Távora <joaotavora <at> gmail.com>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Fri, 28 Aug 2020 10:55:39 -0700
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> In semantic-idle-summary-idle-function:
> cedet/semantic/idle.el:771:10: Warning: ‘eldoc-message’ is an obsolete
>     function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.
>
> In semantic-idle-summary-refresh-echo-area:
> cedet/semantic/idle.el:802:13: Warning: ‘eldoc-message’ is an obsolete
>     function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.
>
> I guess this is the result of João's changes in July?  I had a quick
> peek at how to fix it, but it wasn't immediately obvious to me.

Copying in João Távora in the hope that the fix will be immediately
obvious to him.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Fri, 28 Aug 2020 22:33:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 42777 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Fri, 28 Aug 2020 23:32:36 +0100
Stefan Kangas <stefan <at> marxist.se> writes:

> Lars Ingebrigtsen <larsi <at> gnus.org> writes:
>
>> In semantic-idle-summary-idle-function:
>> cedet/semantic/idle.el:771:10: Warning: ‘eldoc-message’ is an obsolete
>>     function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.
>>
>> In semantic-idle-summary-refresh-echo-area:
>> cedet/semantic/idle.el:802:13: Warning: ‘eldoc-message’ is an obsolete
>>     function (as of eldoc-1.1.0); use ‘eldoc-documentation-functions’ instead.
>>
>> I guess this is the result of João's changes in July?  I had a quick
>> peek at how to fix it, but it wasn't immediately obvious to me.
>
> Copying in João Távora in the hope that the fix will be immediately
> obvious to him.

Lars, you want to fix the compilation warning or something else?  The
function is obsolete, but it leep working.  The only "fix" is to follow
the advice in the warning message.  I don't know why this particular
compilation warning didn't pop up earlier, I don't recall it.

João




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 13:48:01 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 15:46:51 +0200
João Távora <joaotavora <at> gmail.com> writes:

>>> In semantic-idle-summary-refresh-echo-area:
>>> cedet/semantic/idle.el:802:13: Warning: ‘eldoc-message’ is an obsolete
>>>     function (as of eldoc-1.1.0); use
>>> ‘eldoc-documentation-functions’ instead.
>>>
>>> I guess this is the result of João's changes in July?  I had a quick
>>> peek at how to fix it, but it wasn't immediately obvious to me.
>>
>> Copying in João Távora in the hope that the fix will be immediately
>> obvious to him.
>
> Lars, you want to fix the compilation warning or something else?  The
> function is obsolete, but it leep working.  The only "fix" is to follow
> the advice in the warning message.

If the function is obsolete, all in-tree callers should be fixed.  I'm
not familiar enough with eldoc to do so, though.

> I don't know why this particular compilation warning didn't pop up
> earlier, I don't recall it.

It's been there ever since you obsoleted eldoc-message, of course.  :-)

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 13:59:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 15:58:13 +0200
João Távora <joaotavora <at> gmail.com> writes:

> ...unless that code is used in another non emacs-master context. I 
> seem to recall that having happened with Gnus once.  Let's hope
> it's not the case of CEDET, in which case the solution is to shoosh
> the byte-compiler in that particular spot.

CEDET is mostly in-tree these days.

> This is odd because I did get a line from Eli to fix some compilation 
> after my changes, and these were not among them.  I wonder if this file
> is always byte-compiled.

It is, but perhaps Eli didn't do a "make bootstrap" to get all the
warnings.  (The CEDET files don't change that much these days, so
they're rarely re-compiled.)

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 14:28:02 GMT) Full text and rfc822 format available.

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, stefan <at> marxist.se, joaotavora <at> gmail.com
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 17:27:35 +0300
> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Date: Sun, 30 Aug 2020 15:58:13 +0200
> Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
> 
> perhaps Eli didn't do a "make bootstrap" to get all the warnings.

I never bootstrap, not on my main development system.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 14:29:01 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 14:54:29 +0100
[Message part 1 (text/plain, inline)]
On Sun, Aug 30, 2020 at 2:47 PM Lars Ingebrigtsen <larsi <at> gnus.org> wrote:


> > Lars, you want to fix the compilation warning or something else?  The
> > function is obsolete, but it leep working.  The only "fix" is to follow
> > the advice in the warning message.
>
> If the function is obsolete, all in-tree callers should be fixed.


...unless that code is used in another non emacs-master context. I
seem to recall that having happened with Gnus once.  Let's hope
it's not the case of CEDET, in which case the solution is to shoosh
the byte-compiler in that particular spot.

I'm
> not familiar enough with eldoc to do so, though.
>

No problem, I'll do it.  It's not hard, though, and the docstrings
should guide anyone to do the same.


> > I don't know why this particular compilation warning didn't pop up
> > earlier, I don't recall it.
>
> It's been there ever since you obsoleted eldoc-message, of course.  :-)
>

This is odd because I did get a line from Eli to fix some compilation
after my changes, and these were not among them.  I wonder if this file
is always byte-compiled.

Thanks,
João
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 14:31:01 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 15:23:13 +0100
[Message part 1 (text/plain, inline)]
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

>> This is odd because I did get a line from Eli to fix some compilation 
>> after my changes, and these were not among them.  I wonder if this file
>> is always byte-compiled.
>
> It is, but perhaps Eli didn't do a "make bootstrap" to get all the
> warnings.  (The CEDET files don't change that much these days, so
> they're rarely re-compiled.)

Anyway, I just had a look at this.  This isn't the normal situation of
an ElDoc user switching from the old "call eldoc-message directly"
protocol to the new callback-based protocol.

Rather, idle.el is mostly a reimplementation of ElDoc itself which uses
a substantial amount of ElDoc internals -- and repeats much of its code.
It is, in my opinion a file to be itself obsoleted.  Anyway, the
solution with less risk here is to use with-supressed-warnings, which is
what the attached patch does.

João

[0001-Suppress-ElDoc-compiler-warnings-in-lisp-cedet-seman.patch (text/x-diff, inline)]
From 2d951fbe6d5af21c839e2ff7ab89aeabff945621 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=A3o=20T=C3=A1vora?= <joaotavora <at> gmail.com>
Date: Sun, 30 Aug 2020 15:19:08 +0100
Subject: [PATCH] Suppress ElDoc compiler warnings in
 lisp/cedet/semantic/idle.el

Fixes: bug#42777

* lisp/cedet/semantic/idle.el (semantic-idle-summary-refresh-echo-area)
(semantic-idle-summary-idle-function): Use `with-supressed-warnings`.
---
 lisp/cedet/semantic/idle.el | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/lisp/cedet/semantic/idle.el b/lisp/cedet/semantic/idle.el
index 76218249c5..8f58f53b14 100644
--- a/lisp/cedet/semantic/idle.el
+++ b/lisp/cedet/semantic/idle.el
@@ -768,7 +768,10 @@ semantic-idle-summary-idle-function
 	    (when (> strlen ea-width)
 	      (setq str (substring str 0 ea-width)))))
 	;; Display it
-        (eldoc-message str))))
+        ;; FIXME: stop using ElDoc's internals and just use ElDoc.
+        (with-suppressed-warnings
+            ((obsolete eldoc-message))
+          (eldoc-message str)))))
 
 (define-minor-mode semantic-idle-summary-mode
   "Toggle Semantic Idle Summary mode.
@@ -799,7 +802,10 @@ semantic-idle-summary-refresh-echo-area
 		(not (and (boundp 'edebug-active) edebug-active))
 		(not cursor-in-echo-area)
 		(not (eq (selected-window) (minibuffer-window))))
-           (eldoc-message eldoc-last-message)
+           ;; FIXME: stop using ElDoc's internals and just use ElDoc.
+           (with-suppressed-warnings
+               ((obsolete eldoc-message))
+             (eldoc-message eldoc-last-message))
          (setq eldoc-last-message nil))))
 
 (semantic-add-minor-mode 'semantic-idle-summary-mode "")
-- 
2.25.1


Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 14:37:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 15:10:38 +0100
[Message part 1 (text/plain, inline)]
On Sun, Aug 30, 2020 at 2:58 PM Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

>
> It is, but perhaps Eli didn't do a "make bootstrap" to get all the
> warnings.  (The CEDET files don't change that much these days, so
> they're rarely re-compiled.)
>

Maybe also related to the fact the function is forward declared in
the file you pointed to.

João
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 14:57:02 GMT) Full text and rfc822 format available.

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

From: Stefan Kangas <stefan <at> marxist.se>
To: João Távora <joaotavora <at> gmail.com>, 
 Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 07:56:30 -0700
João Távora <joaotavora <at> gmail.com> writes:

> Rather, idle.el is mostly a reimplementation of ElDoc itself which uses
> a substantial amount of ElDoc internals -- and repeats much of its code.
> It is, in my opinion a file to be itself obsoleted.

I'm not familiar with idle.el, but I'd suggest to open a new bug report
to track that work.

Best regards,
Stefan Kangas




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 15:47:01 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 17:46:10 +0200
João Távora <joaotavora <at> gmail.com> writes:

> Anyway, the solution with less risk here is to use
> with-supressed-warnings, which is what the attached patch does.

Hm...  I don't think that's a good idea.  We shouldn't suppress warnings
that point to actual problems, like this one does: eldoc-message will be
going away some day, and then idle.el will stop working (without any
warning).

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 16:01:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 17:00:07 +0100
[Message part 1 (text/plain, inline)]
On Sun, Aug 30, 2020, 16:46 Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> João Távora <joaotavora <at> gmail.com> writes:
>
> > Anyway, the solution with less risk here is to use
> > with-supressed-warnings
>

Hm...  I don't think that's a good idea.  We shouldn't suppress warnings
> that point to actual problems, like this one does: eldoc-message will be
> going away some day, and then idle.el will stop working (without any
> warning).



Not without _any warning_. I'm only supressing the obsoletion warning. If
you remove a function you should in principle get another warning.

João
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 16:03:01 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 18:01:53 +0200
João Távora <joaotavora <at> gmail.com> writes:

> Not without _any warning_. I'm only supressing the obsoletion warning. If you
> remove a function you should in principle get another warning.

That's true, but only after removing it.

Suppressing obsoletion warnings in in-tree code is counter-productive,
in my opinion.  (The exception is code in lisp/obsolete, of course.)

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Sun, 30 Aug 2020 19:24:01 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Sun, 30 Aug 2020 20:22:32 +0100
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> João Távora <joaotavora <at> gmail.com> writes:
>
>> Not without _any warning_. I'm only supressing the obsoletion warning. If you
>> remove a function you should in principle get another warning.
>
> That's true, but only after removing it.
>
> Suppressing obsoletion warnings in in-tree code is counter-productive,
> in my opinion.  (The exception is code in lisp/obsolete, of course.)

That's true.  But look, the real underlying problem we have in our code
that there is code in CEDET's idle.el that needlessly duplicates and
uses implementation detail of the ElDoc infrastructure.  The real fix to
this bug is _integration_, and integration is hard and takes time. 

In parallel, in my opinion, there is little point in having 0 warnings
globally.  Maybe that is the policy in the Emacs repo, but in my opinion
I'd rather investigate "positive deltas" in warning count.  The warnings
in this case point and mark the problem described above.  We could leave
them until someone (maybe CEDET's author) either does the integration
work, or moves idle.el to lisp/obsolete.

João





Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Tue, 01 Sep 2020 14:15:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Tue, 01 Sep 2020 16:14:00 +0200
João Távora <joaotavora <at> gmail.com> writes:

> That's true.  But look, the real underlying problem we have in our code
> that there is code in CEDET's idle.el that needlessly duplicates and
> uses implementation detail of the ElDoc infrastructure.  The real fix to
> this bug is _integration_, and integration is hard and takes time. 

Indeed.  And meanwhile the warning should stay.

> In parallel, in my opinion, there is little point in having 0 warnings
> globally.  Maybe that is the policy in the Emacs repo, but in my opinion
> I'd rather investigate "positive deltas" in warning count.

No, that's not the policy, as far as I know: Warnings stay in until the
problem is fixed, one way or another.

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Tue, 01 Sep 2020 18:23:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Tue, 01 Sep 2020 19:22:19 +0100
Lars Ingebrigtsen <larsi <at> gnus.org> writes:
> João Távora <joaotavora <at> gmail.com> writes:
>
>> That's true.  But look, the real underlying problem we have in our code
>> that there is code in CEDET's idle.el that needlessly duplicates and
>> uses implementation detail of the ElDoc infrastructure.  The real fix to
>> this bug is _integration_, and integration is hard and takes time. 
> Indeed.  And meanwhile the warning should stay.

We agree then.

>> In parallel, in my opinion, there is little point in having 0 warnings
>> globally.  Maybe that is the policy in the Emacs repo, but in my opinion
>> I'd rather investigate "positive deltas" in warning count.
>
> No, that's not the policy, as far as I know: Warnings stay in until the
> problem is fixed, one way or another.

That's my view as well.  So what's missing perhaps is that we close this
bug and open another bug, as was Stefan Kangas' suggestion, to track the
integration work.  WDYT?  Or maybe we should just leave this bug as that
record, since it's where all the investigation done so far is
registered anyway.

Joãp




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Tue, 01 Sep 2020 18:58:01 GMT) Full text and rfc822 format available.

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

From: Stefan Kangas <stefan <at> marxist.se>
To: João Távora <joaotavora <at> gmail.com>, 
 Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Tue, 1 Sep 2020 11:57:49 -0700
João Távora <joaotavora <at> gmail.com> writes:

> That's my view as well.  So what's missing perhaps is that we close this
> bug and open another bug, as was Stefan Kangas' suggestion, to track the
> integration work.  WDYT?  Or maybe we should just leave this bug as that
> record, since it's where all the investigation done so far is
> registered anyway.

I don't have any strong feelings about it, but I think we could keep
this bug report as is.

My main concern was that this piece of information would be buried in a
long sub-thread of an unrelated issue.  But that doesn't seem to be the
case here if obsoleting that file is indeed the solution we're going for.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Tue, 01 Sep 2020 23:44:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 42777 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Wed, 2 Sep 2020 00:43:09 +0100
[Message part 1 (text/plain, inline)]
On Tue, Sep 1, 2020, 19:57 Stefan Kangas <stefan <at> marxist.se> wrote:

I don't have any strong feelings about it, but I think we could keep
> this bug report as is.
>
> My main concern was that this piece of information would be buried in a
> long sub-thread of an unrelated issue.  But that doesn't seem to be the
> case here if obsoleting that file is indeed the solution we're going for.
>

I agree, though I wouldn't yet go as far as asserting with 100% confidence
that obsoleting idle.el entirely is what should be done, though it's a
possibility. Just that we have an integration problem with duplicate
functionality, duplicate code and shared internals. Maybe just fixing the
later is enough. Because other than that, I feel the whole of CEDET has
been pretty obsolete, de facto, for a while now.

João

>
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Wed, 02 Sep 2020 13:39:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Wed, 02 Sep 2020 15:38:17 +0200
João Távora <joaotavora <at> gmail.com> writes:

> Maybe just fixing the later is enough. Because other than that, I feel
> the whole of CEDET has been pretty obsolete, de facto, for a while
> now.

This is somewhat off-topic, but I wonder why.  I mean, I don't use it
myself, but I've tested it a bit now and then when fixing compilation
bugs and stuff, and it seems very cool.  It does a lot of what the
now-very-popular LSP stuff does, but still nobody seems to give CEDET
any love these days...

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Wed, 02 Sep 2020 18:47:02 GMT) Full text and rfc822 format available.

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

From: João Távora <joaotavora <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Wed, 2 Sep 2020 19:46:02 +0100
[Message part 1 (text/plain, inline)]
On Wed, Sep 2, 2020, 14:38 Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> João Távora <joaotavora <at> gmail.com> writes:
>
> > Maybe just fixing the later is enough. Because other than that, I feel
> > the whole of CEDET has been pretty obsolete, de facto, for a while
> > now.
>
> This is somewhat off-topic, but I wonder why.  I mean, I don't use it
> myself, but I've tested it a bit now and then when fixing compilation
> bugs and stuff, and it seems very cool.  It does a lot of what the
> now-very-popular LSP stuff does, but still nobody seems to give CEDET
> any love these days...


CEDET was promising, I followed it closely around 2007-2011 but it never
delivered, IME. I remember being very frustrated by the inability to set up
a simple "find the definition". Some awkward thing called "senator" had to
be built, I think. Many (re)inventions there, as there were elsewhere (i.e.
SLIME): Emacs simply didn't have the infrastructure of Eldoc, Xref,
Flymake, etc that it does now.

Also it suffers from the problem of not being LSP :) CEDET is an
Emacs-specific protocol and an Emacs-specific toolkit to develop
language-specific tools that speak that protocol. LSP is an editor agnostic
and language agnostic protocol only. A very clever idea. You need
relatively little effort on both sides to get something really useful
going. And any effort you spend on one side is worth n-fold on the other
side. That is its winning formula.

João
[Message part 2 (text/html, inline)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Fri, 04 Sep 2020 02:24:01 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Fri, 04 Sep 2020 04:23:17 +0200
João Távora <joaotavora <at> gmail.com> writes:

> Also it suffers from the problem of not being LSP :) CEDET is an
> Emacs-specific protocol and an Emacs-specific toolkit to develop
> language-specific tools that speak that protocol. LSP is an editor
> agnostic and language agnostic protocol only.  A very clever idea. You
> need relatively little effort on both sides to get something really
> useful going. And any effort you spend on one side is worth n-fold on
> the other side. That is its winning formula.

Yeah, that's true.

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




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#42777; Package emacs. (Tue, 06 Jul 2021 15:22:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 42777 <at> debbugs.gnu.org, Stefan Kangas <stefan <at> marxist.se>
Subject: Re: bug#42777: 28.0.50; Obsolete eldoc-message function used in CEDET
Date: Tue, 06 Jul 2021 17:21:02 +0200
This was fixed recently, I believe, so I'm closing this bug report.

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





bug marked as fixed in version 28.1, send any further explanations to 42777 <at> debbugs.gnu.org and Lars Ingebrigtsen <larsi <at> gnus.org> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Tue, 06 Jul 2021 15:22:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 04 Aug 2021 11:24:08 GMT) Full text and rfc822 format available.

This bug report was last modified 2 years and 258 days ago.

Previous Next


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