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

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

bug#59490: 29.0.50; Add C# support


From: Theodor Thornhill
Subject: bug#59490: 29.0.50; Add C# support
Date: Wed, 23 Nov 2022 22:25:12 +0100


On 23 November 2022 22:19:21 CET, Yuan Fu <casouri@gmail.com> wrote:
>
>
>> On Nov 23, 2022, at 12:43 PM, Theodor Thornhill <theo@thornhill.no> wrote:
>> 
>> 
>> 
>> On 23 November 2022 21:38:19 CET, Yuan Fu <casouri@gmail.com> wrote:
>>> 
>>> Yuan Fu <casouri@gmail.com> writes:
>>> 
>>>> Eli Zaretskii <eliz@gnu.org> writes:
>>>> 
>>>>>> Date: Tue, 22 Nov 2022 22:47:12 +0100
>>>>>> From:  Theodor Thornhill via "Bug reports for GNU Emacs,
>>>>>> the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
>>>>>> 
>>>>>> Ok so this works.
>>>>>> 
>>>>>> Sorry for the mess.
>>>>>> 
>>>>>> What do you think?
>>>>> 
>>>>> LGTM, thanks.  I'll let Yuan review and install.
>>>> 
>>>> Thanks Theo! I’ve merged it. One general comment is that we should have
>>>> a "standard" list of features consistent across major modes. I think by
>>>> now we’ve worked on font-lock enough to know what those features should
>>>> be. I’ll start a discussion on the ML.
>>> 
>>> Actually, another thing: it seems the mode defined is still named
>>> csharp-mode rather than csharp-cc-mode.
>>> 
>>> Yuan
>> 
>> Yeah that's right. Imo that's ok, because it's unlikely there will be much 
>> new development on the cc mode variant. So it will be identical to the 
>> upstream one, and the upstream one will be marked as deprecated very soon.
>> 
>> What do you think?
>
>Yeah I was just confused because you said
>
>> I made two new modes, 'csharp-cc-mode' and 'csharp-ts-mode'. The cc-mode
>> variant has that name as to not conflict with the ELPA package.

Yep, that was unclear. I decided to skip it in the second patch :)

>
>And I only see csharp-mode.
>
>So the mode we provide in Emacs are csharp-mode and csharp-ts-mode? Then I 
>assume this line is a typo?
>
>;;;###autoload
>(add-to-list 'auto-mode-alist '("\\.cs\\'" . csharp-cc-mode))
>
>Yuan
>

That's correct. That's a typo. My bad :) make it csharp-mode. You want a new 
patch?





reply via email to

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