GNU bug report logs - #65418
29.1; Eglot: support clangd inactiveRegions extension

Previous Next

Full log

View this message in rfc822 format

From: João Távora <joaotavora <at> gmail.com>
To: Filippo Argiolas <filippo.argiolas <at> gmail.com>
Cc: 65418 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>, Felician Nemeth <felician.nemeth <at> gmail.com>, Philip Kaludercic <philipk <at> posteo.net>
Subject: bug#65418: 29.1; Eglot: support clangd inactiveRegions extension
Date: Mon, 4 Sep 2023 11:51:33 +0100

On Mon, Sep 4, 2023 at 5:09 AM Filippo Argiolas
<filippo.argiolas <at> gmail.com> wrote:
>
> On Mon, Sep 4, 2023 at 5:59 AM Filippo Argiolas
> <filippo.argiolas <at> gmail.com> wrote:
> >
> > On Mon, Sep 4, 2023 at 3:06 AM João Távora <joaotavora <at> gmail.com> wrote:
> > >
> > > On Mon, Sep 4, 2023 at 2:05 AM João Távora <joaotavora <at> gmail.com> wrote:
> > > >
> > > > On Thu, Aug 31, 2023 at 6:28 PM Filippo Argiolas
> > > > <filippo.argiolas <at> gmail.com> wrote:
> > > > >
> > > > > On Sun, Aug 27, 2023 at 3:58 PM João Távora <joaotavora <at> gmail.com> wrote:
> > > > > > Glad to help.  You're lucky I'm not some kind of wine connoisseur ;-)
> > > > > > Make a patch for that (and remember to also include the exportation of
> > > > > > the '--' symbols).
> > > > > >
> > > > >
> > > > > Many thanks again! That code definitely made this work week more enjoyable ;-)
> > > > >
> > > > > Sorry for the delay with the patch, little time left these days for
> > > > > side projects or even trivial contributions like this.
> > > > > See if the attached patches could do!
> > > >
> > > > They were very helpful.  I just pushed heavily edited versions of
> > > > them to master.  I used them as a starting point and credited you as
> > > > co-author (I could have done the other way round, crediting myself
> > > >  as co-author, but the changes I did were a bit too extensive for
> > > > that).
> >
> > No problem for me, edit as much as you want! just glad to help.
> > I'd be just as happy if you just thanked me in the commit message.
>
> By the way I looked at your doc version and very much approve it. The
> one about public API I think I don't even deserve co-authorship :-D

It's mainly because i started by applying your patch then doing multiple
changes to them.

My intuition also tells me that the remnants of both your
contribution are indeed "trivial" (as measured both in absolute LOC
and complexity)

But  I admit I'm not fully aware of how these things are counted
"officially", so better safe than sorry and I reverted them.  But
I would very much like to re-revert of course, as this was a lot
of work and is a pretty valuable change, IMO.  Let's wait for Eli to
explain how to proceed.

João

Previous Next

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