From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Paul Pogonyshev Newsgroups: gmane.emacs.bugs Subject: bug#15465: CC-mode misaligns enums that implement an interface in Java Date: Mon, 14 Oct 2013 11:19:39 +0200 Message-ID: References: <20130928173323.GA11317@acm.acm> <20130929150345.GD3161@acm.acm> <20131013213838.GB2498@acm.acm> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=089e013c6e4aad8ae204e8aff72e X-Trace: ger.gmane.org 1381742419 22895 80.91.229.3 (14 Oct 2013 09:20:19 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 14 Oct 2013 09:20:19 +0000 (UTC) Cc: 15465@debbugs.gnu.org To: Alan Mackenzie Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Oct 14 11:20:22 2013 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1VVeK5-000525-3O for geb-bug-gnu-emacs@m.gmane.org; Mon, 14 Oct 2013 11:20:21 +0200 Original-Received: from localhost ([::1]:35923 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VVeK4-0001Tj-KV for geb-bug-gnu-emacs@m.gmane.org; Mon, 14 Oct 2013 05:20:20 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:53933) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VVeJx-0001Ta-Ie for bug-gnu-emacs@gnu.org; Mon, 14 Oct 2013 05:20:18 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VVeJs-0000AK-Si for bug-gnu-emacs@gnu.org; Mon, 14 Oct 2013 05:20:13 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:33814) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VVeJn-0008Sh-KG; Mon, 14 Oct 2013 05:20:03 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1VVeJn-0004sQ-1i; Mon, 14 Oct 2013 05:20:03 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Paul Pogonyshev Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org, bug-cc-mode@gnu.org Resent-Date: Mon, 14 Oct 2013 09:20:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 15465 X-GNU-PR-Package: emacs,cc-mode X-GNU-PR-Keywords: Original-Received: via spool by 15465-submit@debbugs.gnu.org id=B15465.138174238818702 (code B ref 15465); Mon, 14 Oct 2013 09:20:02 +0000 Original-Received: (at 15465) by debbugs.gnu.org; 14 Oct 2013 09:19:48 +0000 Original-Received: from localhost ([127.0.0.1]:47833 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1VVeJY-0004rY-0U for submit@debbugs.gnu.org; Mon, 14 Oct 2013 05:19:48 -0400 Original-Received: from mail-ie0-f177.google.com ([209.85.223.177]:55505) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1VVeJU-0004r9-Nc for 15465@debbugs.gnu.org; Mon, 14 Oct 2013 05:19:45 -0400 Original-Received: by mail-ie0-f177.google.com with SMTP id e14so8668170iej.36 for <15465@debbugs.gnu.org>; Mon, 14 Oct 2013 02:19:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=1FqUP/LXcAVZMVhksgZp8oLMvbYb9pJtJ7D6XvWtsmM=; b=057iB+3tb3SDVWZriua42RddyT3q74BflwH6RXzMy6Kk6CNeb2ZNOq1LmHSYsJo/TI rvErkJKPB4otgdZUP1MAFosGO7Oq0CBHDTeIoGT0BqsHFQLVFeZwWZ8fvdkC0S7TKcl6 tN8asDJVtmTmb/tlp9Xq5Y6Y+TvhIUKs/HImQ7FtGwxB/CuyFtnkyJy8hq/miwdGrhsq T+Tt8nrVtzF+g1RHf663nyuxs4A6cBsnSJlkH9iwUERW81ZveYDvIEKDJfU9xGCot+zi Nnv6x569oORm1V4QhfWPhbx739dLqv/ucRxf5/UFY959fjgzIS5lYNeSqqHT+j6J7/MJ 5k5g== X-Received: by 10.50.119.4 with SMTP id kq4mr12380176igb.40.1381742379106; Mon, 14 Oct 2013 02:19:39 -0700 (PDT) Original-Received: by 10.42.240.66 with HTTP; Mon, 14 Oct 2013 02:19:39 -0700 (PDT) In-Reply-To: <20131013213838.GB2498@acm.acm> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:79236 Archived-At: --089e013c6e4aad8ae204e8aff72e Content-Type: text/plain; charset=UTF-8 Um, it seems to be worse than before. I guess your first patch worked better, at least for my cases. E.g. now I have: public enum X implements Y, Z { A, B; } On a side note, enum methods (and constructors --- yes, enums can optionally have those) are fontified as variable names, though that is of course less important than indentation. Paul On 13 October 2013 23:38, Alan Mackenzie wrote: > Hello, Paul. > > On Mon, Sep 30, 2013 at 09:23:53AM +0200, Paul Pogonyshev wrote: > > Sorry, it doesn't apply. > > I don't understand what's gone wrong here. Anyway, I've just committed > that patch to the Emacs trunk (revision #114650). Perhaps you could try > out this latest fix too, and let me know if there are any more problems > in this area. Thanks! > > > Paul > > -- > Alan Mackenzie (Nuremberg, Germany). > > > > On 29 September 2013 17:03, Alan Mackenzie wrote: > > > > > On Sat, Sep 28, 2013 at 09:15:52PM +0200, Paul Pogonyshev wrote: > > > > > I've committed a fix, revision #114474, to the bzr trunk. Could > > > > > you try out the change, please, and either confirm it fixes the > > > > > bug properly, or tell me what hasn't yet been fixed. > > > > > > > Thank you. The case I initially mentioned works properly now, also > with > > > > multiple implemented interfaces. However, it seems to not know about > > > > interfaces with generics: > > > > > > > public enum X implements Y > > > > { > > > > A, > > > > B; > > > > } > > > > > > Here's a better patch than the one from last night, which was > > > demonstrably buggy. It should apply cleanly to the current trunk. > > > Please let me know how thoroughly it works. > > > > > > > --089e013c6e4aad8ae204e8aff72e Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Um, it seems to be worse than before. I guess your fi= rst patch worked better, at least for my cases.

E.g. now I have:
=
public enum X implements Y, Z
{
=C2=A0=C2=A0=C2=A0 A,
=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 B;
}

On a side note, enum methods (and constructors --- yes, enums ca= n optionally have those) are fontified as variable names, though that is of= course less important than indentation.

Paul


On 13 O= ctober 2013 23:38, Alan Mackenzie <acm@muc.de> wrote:
Hello, Paul.

On Mon, Sep 30, 2013 at 09:23:53AM +0200, Paul Pogonyshev wrote:
> Sorry, it doesn't apply.

I don't understand what's gone wrong here. =C2=A0Anyway, I= 9;ve just committed
that patch to the Emacs trunk (revision #114650). =C2=A0Perhaps you could t= ry
out this latest fix too, and let me know if there are any more problems
in this area. =C2=A0Thanks!

> Paul

--
Alan Mackenzie (Nuremberg, Germany).


> On 29 September 2013 17:03, Alan Macken= zie <acm@muc.de> wrote:
>
> > On Sat, Sep 28, 201= 3 at 09:15:52PM +0200, Paul Pogonyshev wrote:
> > > > I've committed a fix, revision #114474, to the bzr = trunk. =C2=A0Could
> > > > you try out the change, please, and either confirm it f= ixes the
> > > > bug properly, or tell me what hasn't yet been fixed= .
> >
> > > Thank you. The case I initially mentioned works properly now= , also with
> > > multiple implemented interfaces. However, it seems to not kn= ow about
> > > interfaces with generics:
> >
> > > public enum X implements Y <Z>
> > > {
> > > =C2=A0 =C2=A0 A,
> > > =C2=A0 =C2=A0 =C2=A0 =C2=A0 B;
> > > }
> >
> > Here's a better patch than the one from last night, which was=
> > demonstrably buggy. =C2=A0It should apply cleanly to the current = trunk.
> > Please let me know how thoroughly it works.
> >
> >

--089e013c6e4aad8ae204e8aff72e--