GNU bug report logs - #59490
29.0.50; Add C# support

Previous Next

Full log

View this message in rfc822 format

From: Theodor Thornhill <theo <at> thornhill.no>
To: Yuan Fu <casouri <at> gmail.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 59490 <at> debbugs.gnu.org
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 <at> gmail.com> wrote:
>
>
>> On Nov 23, 2022, at 12:43 PM, Theodor Thornhill <theo <at> thornhill.no> wrote:
>> 
>> 
>> 
>> On 23 November 2022 21:38:19 CET, Yuan Fu <casouri <at> gmail.com> wrote:
>>> 
>>> Yuan Fu <casouri <at> gmail.com> writes:
>>> 
>>>> Eli Zaretskii <eliz <at> 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 <at> 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?

Previous Next

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