Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update clang-format to clang-format-11, fixes enum bug #1287

Merged
merged 1 commit into from
Mar 20, 2021

Conversation

ailin-nemui
Copy link
Contributor

No description provided.

@ailin-nemui ailin-nemui added the auto-merge This PR is scheduled for merge if no further comments are opened label Mar 14, 2021
@ailin-nemui
Copy link
Contributor Author

@irssi/developers

@ailin-nemui
Copy link
Contributor Author

Example of the bug:

diff --git a/src/fe-text/textbuffer.h b/src/fe-text/textbuffer.h
index 5f5f9a2..a8b230a 100644
--- a/src/fe-text/textbuffer.h
+++ b/src/fe-text/textbuffer.h
@@ -11,11 +11,10 @@ enum {
 	LINE_CMD_EOL=0x80,	/* line ends here */
 };
 
-enum {
-	COLORING_STRIP = 0,
-	COLORING_EXPAND = 1,
-	COLORING_UNEXPAND = 2,
-	COLORING_RAW = 3,
+enum { COLORING_STRIP = 0,
+       COLORING_EXPAND = 1,
+       COLORING_UNEXPAND = 2,
+       COLORING_RAW = 3,
 };
 
 struct _TEXT_BUFFER_FORMAT_REC;

https://github.com/irssi/irssi/runs/2107467912?check_suite_focus=true

@ailin-nemui ailin-nemui merged commit 6f38095 into irssi:master Mar 20, 2021
@ailin-nemui ailin-nemui deleted the clang-enum branch March 20, 2021 18:02
@ailin-nemui ailin-nemui added this to the 1.3.0 milestone Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge This PR is scheduled for merge if no further comments are opened
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant