public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug modula2/108485] New: CppArg is broken for whitespaces
@ 2023-01-20 20:18 pinskia at gcc dot gnu.org
  2023-08-12 20:04 ` [Bug modula2/108485] " gaius at gcc dot gnu.org
  0 siblings, 1 reply; 2+ messages in thread
From: pinskia at gcc dot gnu.org @ 2023-01-20 20:18 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108485

            Bug ID: 108485
           Summary: CppArg is broken for whitespaces
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: modula2
          Assignee: gaius at gcc dot gnu.org
          Reporter: pinskia at gcc dot gnu.org
  Target Milestone: ---

Just quickly looking into CppArg and I noticed the arguments are not quoted or
otherwise fixed up when doing ConCatChar.
This means if you had a whitespace in one of the options, the options become
two because it gets reinterrupted.

The answer is to always just pass around an array of arguments and never do
concatting like this but this is a huge change ...

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

* [Bug modula2/108485] CppArg is broken for whitespaces
  2023-01-20 20:18 [Bug modula2/108485] New: CppArg is broken for whitespaces pinskia at gcc dot gnu.org
@ 2023-08-12 20:04 ` gaius at gcc dot gnu.org
  0 siblings, 0 replies; 2+ messages in thread
From: gaius at gcc dot gnu.org @ 2023-08-12 20:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108485

Gaius Mulley <gaius at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2023-08-12
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |ASSIGNED

--- Comment #1 from Gaius Mulley <gaius at gcc dot gnu.org> ---
Indeed - thanks for spotting and reporting the bug.

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

end of thread, other threads:[~2023-08-12 20:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-20 20:18 [Bug modula2/108485] New: CppArg is broken for whitespaces pinskia at gcc dot gnu.org
2023-08-12 20:04 ` [Bug modula2/108485] " gaius at gcc dot gnu.org

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).