GNU bug report logs - #21799
Requesting review for change to js.el

Previous Next

Package: emacs;

Reported by: Jackson Hamilton <jackson <at> jacksonrayhamilton.com>

Date: Sat, 31 Oct 2015 09:39:02 UTC

Severity: wishlist

Tags: patch

Done: Dmitry Gutov <dgutov <at> yandex.ru>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Jackson Hamilton <jackson <at> jacksonrayhamilton.com>
Subject: bug#21799: closed (Re: bug#21799: Requesting review for change to
 js.el)
Date: Tue, 23 Feb 2016 10:15:03 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#21799: Requesting review for change to js.el

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 21799 <at> debbugs.gnu.org.

-- 
21799: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=21799
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Lars Ingebrigtsen <larsi <at> gnus.org>,
 Jackson Hamilton <jackson <at> jacksonrayhamilton.com>
Cc: 21799-done <at> debbugs.gnu.org
Subject: Re: bug#21799: Requesting review for change to js.el
Date: Tue, 23 Feb 2016 12:14:35 +0200
On 02/23/2016 07:15 AM, Lars Ingebrigtsen wrote:

> Looks good to me, I think, but I think it should probably be separated
> out into its own file: jsx-mode.el.

Sorry, we've already applied it and forgot to close the bug.

Calling it jsx-mode.el might be suboptimal due to the conflict: 
https://github.com/jsx/jsx-mode.el (and that one's for a different JSX).

> Also, I can't find your name in the Emacs copyright assignment file.

It's there, as "Jackson Ray Hamilton".

[Message part 3 (message/rfc822, inline)]
From: Jackson Hamilton <jackson <at> jacksonrayhamilton.com>
To: "bug-gnu-emacs <at> gnu.org" <bug-gnu-emacs <at> gnu.org>
Subject: Requesting review for change to js.el
Date: Sat, 31 Oct 2015 02:37:33 -0700
[Message part 4 (text/plain, inline)]
See attached patch. Adds JSX indentation support to js-mode.
[Message part 5 (text/html, inline)]
[jsx.patch (text/x-patch, attachment)]

This bug report was last modified 9 years and 142 days ago.

Previous Next


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