[Bug d/99917] gcc/d/dmd/mtype.c:5223: missing call to va_end ?

2021-04-06 Thread ibuclaw at gdcproject dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99917

Iain Buclaw  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #3 from Iain Buclaw  ---
Fix committed.

[Bug d/99917] gcc/d/dmd/mtype.c:5223: missing call to va_end ?

2021-04-06 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99917

--- Comment #2 from CVS Commits  ---
The master branch has been updated by Iain Buclaw :

https://gcc.gnu.org/g:d48f87d5c1927b1bf2009af3251fe8757e823713

commit r11-8016-gd48f87d5c1927b1bf2009af3251fe8757e823713
Author: Iain Buclaw 
Date:   Tue Apr 6 13:12:54 2021 +0200

d: Fix missing call to va_end in getMatchError [PR99917]

Reviewed-on: https://github.com/dlang/dmd/pull/12380

gcc/d/ChangeLog:

PR d/99917
* dmd/MERGE: Merge upstream dmd d16195406.

[Bug d/99917] gcc/d/dmd/mtype.c:5223: missing call to va_end ?

2021-04-05 Thread ibuclaw at gdcproject dot org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99917

--- Comment #1 from Iain Buclaw  ---
(In reply to David Binderman from comment #0)
> trunk.git/gcc/d/dmd/mtype.c:5223:30: error: va_list 'ap' was opened but not
> closed by va_end(). [va_end_missing]
> 
> Source code is
> 
> va_list ap;
> va_start(ap, format);
> buf.vprintf(format, ap);
> return buf.extractChars();

Had a look and confirmed, raised a change in upstream dmd and awaiting for it
to be reviewed.