[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [FR] Allow flattened imenu index
From: |
João Távora |
Subject: |
Re: [FR] Allow flattened imenu index |
Date: |
Mon, 11 Dec 2023 23:57:16 +0000 |
On Mon, Dec 11, 2023 at 11:54 PM Dmitry Gutov <dmitry@gutov.dev> wrote:
>
> On 12/12/2023 01:48, João Távora wrote:
> > On Mon, Dec 11, 2023 at 11:41 PM Dmitry Gutov<dmitry@gutov.dev> wrote:
> >> On 12/12/2023 01:35, João Távora wrote:
> >>> Or maybe say what the ruby-mode imenu backend
> >>> does?
> >> Juri just posted a long example of what ruby-mode does (and ruby-ts-mode
> >> too) in his last message in this thread.
> > OK, and doesn't it answer_your_ question? With Eglot
> >
> > Method > ModuleExample::ClassExample > instance_method
> > Method > ClassExample > class_method
> >
> > If both methods were named "foo" no problem, right?
>
> But this doesn't look right: if 'class_method' is in a class called
> ModuleExample::ClassExample, shouldn't its entry look like
>
> Method > ModuleExample::ClassExample > class_method
>
> as well?
Take it up with the LS. I didn't write it.
> And then, if both 'instance_method' and 'class_method' actually are
> called 'foo', then we'd have a problem.
Bizarrely, you wouldn't have a tree to start with, which probably
explains why the devs of the LS Juri is using opted for that
layout.
> This entry from his message also looks odd:
>
> Module > > ModuleExample
If you can reproduce it, show the repro.
João
- Re: [FR] Allow flattened imenu index, (continued)
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/11
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/11
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/11
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/11
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/11
- Re: [FR] Allow flattened imenu index,
João Távora <=
- Re: [FR] Allow flattened imenu index, Ihor Radchenko, 2023/12/11
- Re: [FR] Allow flattened imenu index, João Távora, 2023/12/11
- Re: [FR] Allow flattened imenu index, Spencer Baugh, 2023/12/15
- Re: [FR] Allow flattened imenu index, Dmitry Gutov, 2023/12/15