From: Carlos Maniero <carlos@maniero.me>
To: ~johnnyrichard/olang-devel@lists.sr.ht
Cc: Carlos Maniero <carlos@maniero.me>
Subject: [PATCH olang 0/6] Suggestions in code style
Date: Thu, 10 Oct 2024 01:33:23 +0000 (UTC) [thread overview]
Message-ID: <20241010013318.222905-1-carlos@maniero.me> (raw)
I split it into small changes so it is possible to be avaliated case by
case. Since this is a very opinionated and is more about preference than
anything, just let me know which changes you agree on and I send a new
patch.
Carlos Maniero (6):
codestyle: change AlignAfterOpenBracket to BlockIndent
codestyle: never BreakBeforeBraces
codestyle: prevent extra empty lines at EOF
codestyle: do not allow single line enums
codestyle: add trailing comma on struct initializer
codestyle: limit the code to 80 characters
.clang-format | 27 +-
src/arena.c | 15 +-
src/arena.h | 3 +-
src/ast.c | 122 ++++---
src/ast.h | 132 ++++----
src/checker.c | 28 +-
src/checker.h | 3 +-
src/cli.c | 55 +--
src/cli.h | 9 +-
src/codegen_linux_aarch64.c | 9 +-
src/codegen_linux_x86_64.c | 615 ++++++++++++++++++++++++----------
src/codegen_linux_x86_64.h | 8 +-
src/lexer.c | 164 +++++----
src/lexer.h | 18 +-
src/list.c | 18 +-
src/list.h | 6 +-
src/main.c | 73 ++--
src/map.c | 53 +--
src/map.h | 9 +-
src/parser.c | 179 ++++++----
src/parser.h | 3 +-
src/pretty_print_ast.c | 133 +++++---
src/scope.c | 31 +-
src/scope.h | 6 +-
src/string_view.c | 14 +-
src/string_view.h | 3 +-
src/type.c | 6 +-
src/type.h | 21 +-
tests/unit/arena_test.c | 35 +-
tests/unit/list_test.c | 45 ++-
tests/unit/map_test.c | 35 +-
tests/unit/string_view_test.c | 55 ++-
32 files changed, 1219 insertions(+), 714 deletions(-)
base-commit: d45df92d81f00059ecd1f6478bac7e4511d9fd8d
--
2.46.0
next reply other threads:[~2024-10-10 1:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-10 1:33 Carlos Maniero [this message]
2024-10-10 1:33 ` [PATCH olang v1 1/6] codestyle: change AlignAfterOpenBracket to BlockIndent Carlos Maniero
2024-10-10 1:33 ` [PATCH olang v1 2/6] codestyle: never BreakBeforeBraces Carlos Maniero
2024-10-10 1:33 ` [PATCH olang v1 3/6] codestyle: prevent extra empty lines at EOF Carlos Maniero
2024-10-10 1:33 ` [PATCH olang v1 4/6] codestyle: do not allow single line enums Carlos Maniero
2024-10-10 1:33 ` [PATCH olang v1 5/6] codestyle: add trailing comma on struct initializer Carlos Maniero
2024-10-10 1:34 ` [olang/patches/.build.yml] build success builds.sr.ht
2024-10-10 1:33 ` [PATCH olang v1 6/6] codestyle: limit the code to 80 characters Carlos Maniero
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241010013318.222905-1-carlos@maniero.me \
--to=carlos@maniero.me \
--cc=~johnnyrichard/olang-devel@lists.sr.ht \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.johnnyrichard.com/olang.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox