unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning
@ 2023-01-18 14:09 Mohammed Sadiq
  2023-01-20 21:24 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 5+ messages in thread
From: Mohammed Sadiq @ 2023-01-18 14:09 UTC (permalink / raw)
  To: 60932

enums are not indented with select -> TAB, Example:

enum {
PROP_0,
PROP_NAME,
N_PROPS
};


When trying to indent, I get the following warning in *Messages*:
indent-line-to: Wrong type argument: wholenump, -2


In GNU Emacs 29.0.60 (build 8, x86_64-pc-linux-gnu, GTK+ Version
  3.24.36, cairo version 1.16.0) of 2023-01-18 built on purism
Repository revision: 1798ff5a6636d6b34d23ab2dadb966e94cc57467
Repository branch: emacs-29
Windowing system distributor 'The X.Org Foundation', version 
11.0.12101005
System Description: Debian GNU/Linux bookworm/sid





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning
  2023-01-18 14:09 bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning Mohammed Sadiq
@ 2023-01-20 21:24 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-01-21  6:44   ` Mohammed Sadiq
  0 siblings, 1 reply; 5+ messages in thread
From: Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-01-20 21:24 UTC (permalink / raw)
  To: Mohammed Sadiq; +Cc: 60932

Mohammed Sadiq <sadiq@sadiqpk.org> writes:

> enums are not indented with select -> TAB, Example:
>
> enum {
> PROP_0,
> PROP_NAME,
> N_PROPS
> };
>
>
> When trying to indent, I get the following warning in *Messages*:
> indent-line-to: Wrong type argument: wholenump, -2
>
>
> In GNU Emacs 29.0.60 (build 8, x86_64-pc-linux-gnu, GTK+ Version
>  3.24.36, cairo version 1.16.0) of 2023-01-18 built on purism
> Repository revision: 1798ff5a6636d6b34d23ab2dadb966e94cc57467
> Repository branch: emacs-29
> Windowing system distributor 'The X.Org Foundation', version 11.0.12101005
> System Description: Debian GNU/Linux bookworm/sid


Thanks!

Should be fixed in latest master, closing this :)

Theo





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning
  2023-01-20 21:24 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-01-21  6:44   ` Mohammed Sadiq
  2023-01-21  7:48     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 5+ messages in thread
From: Mohammed Sadiq @ 2023-01-21  6:44 UTC (permalink / raw)
  To: Theodor Thornhill; +Cc: 60932

On 2023-01-21 02:54, Theodor Thornhill wrote:
> Should be fixed in latest master, closing this :)

Well, this should be fixed in emacs-29 branch too, right?
The warning has gone there, but enum indentation is still broken,

I get:

enum {
GTK_PAD_ACTION_BUTTON,
GTK_PAD_ACTION_RING,
GTK_PAD_ACTION_STRIP
} GtkPadActionType;

but I expect to get:

enum {
   GTK_PAD_ACTION_BUTTON,
   GTK_PAD_ACTION_RING,
   GTK_PAD_ACTION_STRIP
} GtkPadActionType;





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning
  2023-01-21  6:44   ` Mohammed Sadiq
@ 2023-01-21  7:48     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2023-01-21  8:00       ` Mohammed Sadiq
  0 siblings, 1 reply; 5+ messages in thread
From: Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2023-01-21  7:48 UTC (permalink / raw)
  To: Mohammed Sadiq; +Cc: 60932

Mohammed Sadiq <sadiq@sadiqpk.org> writes:

> On 2023-01-21 02:54, Theodor Thornhill wrote:
>> Should be fixed in latest master, closing this :)
>
> Well, this should be fixed in emacs-29 branch too, right?
> The warning has gone there, but enum indentation is still broken,
>
> I get:
>
> enum {
> GTK_PAD_ACTION_BUTTON,
> GTK_PAD_ACTION_RING,
> GTK_PAD_ACTION_STRIP
> } GtkPadActionType;
>
> but I expect to get:
>
> enum {
>    GTK_PAD_ACTION_BUTTON,
>    GTK_PAD_ACTION_RING,
>    GTK_PAD_ACTION_STRIP
> } GtkPadActionType;


Sorry, my bad.  Applied to emacs-29 now.  Does it work for you now?

Theo





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning
  2023-01-21  7:48     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2023-01-21  8:00       ` Mohammed Sadiq
  0 siblings, 0 replies; 5+ messages in thread
From: Mohammed Sadiq @ 2023-01-21  8:00 UTC (permalink / raw)
  To: Theodor Thornhill; +Cc: 60932

On 2023-01-21 13:18, Theodor Thornhill wrote:
> Sorry, my bad.  Applied to emacs-29 now.  Does it work for you now?

Yes it does, thanks for fixing.





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-01-21  8:00 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-18 14:09 bug#60932: 29.0.60; c-ts-mode: Fails to indent enum block emitting a warning Mohammed Sadiq
2023-01-20 21:24 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-01-21  6:44   ` Mohammed Sadiq
2023-01-21  7:48     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-01-21  8:00       ` Mohammed Sadiq

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).