public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug go/61746] New: A line starting with /*// causes an error even if the /* is closed on the next line
@ 2014-07-08 12:59 erezh at il dot ibm.com
  2014-07-08 20:50 ` [Bug go/61746] " ian at gcc dot gnu.org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: erezh at il dot ibm.com @ 2014-07-08 12:59 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 61746
           Summary: A line starting with /*// causes an error even if the
                    /* is closed on the next line
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: go
          Assignee: ian at airs dot com
          Reporter: erezh at il dot ibm.com
                CC: cmang at google dot com

Created attachment 33089
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33089&action=edit
The file to compile to produce the error

OS: Ubuntu 14.04 LTS, on either amd64 or ppc64le

gccgo version:
go version xgcc (Ubuntu 4.9-20140406-0ubuntu1) 4.9.0 20140405 (experimental)
[trunk revision 209157] linux/amd64 [or linux/ppc64]

How-to-produce:
Compile the attached file hello.go.Tthe compiler responds with an error as
following:
# gccgo -g -save-temps -o hello hello.go
hello.go:7:4: error: expected declaration
 /*// problem comment here
    ^
hello.go:8:1: error: expected declaration
 */
 ^
hello.go:12:1: error: expected declaration
 }
 ^

Expected-output:
There should be no error. When compiling with golang-go 1.2.2, there isn't an
error produced.


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

* [Bug go/61746] A line starting with /*// causes an error even if the /* is closed on the next line
  2014-07-08 12:59 [Bug go/61746] New: A line starting with /*// causes an error even if the /* is closed on the next line erezh at il dot ibm.com
@ 2014-07-08 20:50 ` ian at gcc dot gnu.org
  2014-07-08 20:50 ` ian at gcc dot gnu.org
  2014-07-08 20:52 ` ian at airs dot com
  2 siblings, 0 replies; 4+ messages in thread
From: ian at gcc dot gnu.org @ 2014-07-08 20:50 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> ---
Author: ian
Date: Tue Jul  8 20:49:59 2014
New Revision: 212371

URL: https://gcc.gnu.org/viewcvs?rev=212371&root=gcc&view=rev
Log:
    PR go/61746
compiler: Fix C-style comment parsing.

The compiler was incorrectly seeing /*/ as a complete C-style
comment.

Modified:
    branches/gcc-4_9-branch/gcc/go/gofrontend/lex.cc


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

* [Bug go/61746] A line starting with /*// causes an error even if the /* is closed on the next line
  2014-07-08 12:59 [Bug go/61746] New: A line starting with /*// causes an error even if the /* is closed on the next line erezh at il dot ibm.com
  2014-07-08 20:50 ` [Bug go/61746] " ian at gcc dot gnu.org
@ 2014-07-08 20:50 ` ian at gcc dot gnu.org
  2014-07-08 20:52 ` ian at airs dot com
  2 siblings, 0 replies; 4+ messages in thread
From: ian at gcc dot gnu.org @ 2014-07-08 20:50 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> ---
Author: ian
Date: Tue Jul  8 20:49:38 2014
New Revision: 212370

URL: https://gcc.gnu.org/viewcvs?rev=212370&root=gcc&view=rev
Log:
    PR go/61746
compiler: Fix C-style comment parsing.

The compiler was incorrectly seeing /*/ as a complete C-style
comment.

Modified:
    trunk/gcc/go/gofrontend/lex.cc


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

* [Bug go/61746] A line starting with /*// causes an error even if the /* is closed on the next line
  2014-07-08 12:59 [Bug go/61746] New: A line starting with /*// causes an error even if the /* is closed on the next line erezh at il dot ibm.com
  2014-07-08 20:50 ` [Bug go/61746] " ian at gcc dot gnu.org
  2014-07-08 20:50 ` ian at gcc dot gnu.org
@ 2014-07-08 20:52 ` ian at airs dot com
  2 siblings, 0 replies; 4+ messages in thread
From: ian at airs dot com @ 2014-07-08 20:52 UTC (permalink / raw)
  To: gcc-bugs

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

Ian Lance Taylor <ian at airs dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Ian Lance Taylor <ian at airs dot com> ---
Fixed.

Thanks for the report.


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

end of thread, other threads:[~2014-07-08 20:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-08 12:59 [Bug go/61746] New: A line starting with /*// causes an error even if the /* is closed on the next line erezh at il dot ibm.com
2014-07-08 20:50 ` [Bug go/61746] " ian at gcc dot gnu.org
2014-07-08 20:50 ` ian at gcc dot gnu.org
2014-07-08 20:52 ` ian at airs dot com

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