From Diogo Santos to ~sircmpwn/email-test-drive
--- This fixes the issues raised from the first patch. dsan | 1 + 1 file changed, 1 insertion(+) create mode 100644 dsan diff --git a/dsan b/dsan new file mode 100644 index 0000000..66cb3d4 --- /dev/null +++ b/dsan @@ -0,0 +1 @@ I have successfully used git send-email! [message trimmed]
From Diogo Santos to ~sircmpwn/email-test-drive
On Sat, 2024-06-01 at 17:41 +0000, git-send-email.io wrote: > Hi Diogo Santos! > > Thanks for the patch! Needs a minor fix, though: > >> diff --git a/dsan b/dsan >> -%<- >> +I'm about to try git send-email > > This statement is no longer correct - you have already tried it, and > succeeded! Can you change this to the following: > > I have successfully used git send-email! >
From Diogo Santos to ~sircmpwn/email-test-drive
--- dsan | 1 + 1 file changed, 1 insertion(+) create mode 100644 dsan diff --git a/dsan b/dsan new file mode 100644 index 0000000..6d0ecfd --- /dev/null +++ b/dsan @@ -0,0 +1 @@ I'm about to try git send-email -- 2.39.5 [message trimmed]
From Diogo Santos to ~mpu/qbe
Yeah, pdpmake is fully capable of generating qbe without any issues. Although not ideal, that was the patch that I came up with. If someone, in the future, can find a cleaner and simpler solution, I am all for it.
From Diogo Santos to ~mpu/qbe
The current makefile isn't compliant with POSIX 2017. Specifically, `pdpmake` warns about: - The presence of '/' in target names - The usage of '#' within commands According to the pdpmake [homepage], the next POSIX standard will address the first issue. [homepage]: https://frippery.org/make/ --- Makefile | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) [message trimmed]
From Diogo Santos to ~mcf/cproc
GCC 13.2 warns about this with `-Wsign-compare`. --- cc.h | 4 ++-- pp.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cc.h b/cc.h index cb53b0d..8defc18 100644 --- a/cc.h +++ b/cc.h @@ -424,9 +424,9 @@ extern enum ppflags ppflags; void ppinit(void); void next(void); [message trimmed]
From Diogo Santos to ~mcf/cproc
Found with the 'include-what-you-use' tool. --- decl.c | 2 -- eval.c | 1 - expr.c | 1 - pp.c | 1 - scan.c | 1 - stmt.c | 3 --- targ.c | 1 - token.c | 1 - utf.h | 3 +++ util.c | 1 - util.h | 3 +++ 11 files changed, 6 insertions(+), 12 deletions(-) [message trimmed]
From Diogo Santos to ~mcf/cproc
Found with clang 18.1.6 using the UndefinedBehaviorSanitizer. Test `compatible-vla-types.c`, line 15, triggers this UB. --- decl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/decl.c b/decl.c index 228c776..7bd37d8 100644 --- a/decl.c +++ b/decl.c @@ -698,7 +698,7 @@ declarator(struct scope *s, struct qualtype base, char **name, struct scope **fu if (e->kind == EXPRCONST) { if (e->type->u.basic.issigned && e->u.constant.u >> 63) error(&tok.loc, "array length must be non-negative"); [message trimmed]
From Diogo Santos to ~mcf/cproc
Found with clang 18.1.6 using UndefinedBehaviorSanitizer. --- pp.c | 3 ++- qbe.c | 16 ++++++++++------ 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/pp.c b/pp.c index 99e56e8..95057b8 100644 --- a/pp.c +++ b/pp.c @@ -523,7 +523,8 @@ expandfunc(struct macro *m) error(&t->loc, "too many arguments for macro '%s'", m->name); for (i = 0, t = tok.val; i < m->nparam; ++i) { arg[i].token = t; [message trimmed]
From Diogo Santos to ~mcf/cproc
Warnings addressed: SC1010, SC2086, and SC2223. --- configure | 22 +++++++++++----------- runtests | 6 +++--- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/configure b/configure index dab1bf3..51572ac 100755 --- a/configure +++ b/configure @@ -30,7 +30,7 @@ for arg ; do esac done [message trimmed]