[Bug c/36367] warning for questionable compound expression

2010-03-31 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-03-31 09:58 ---
Author: ebotcazou
Date: Thu May 29 11:33:17 2008
New Revision: 136154

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=136154
Log:
* tree-nested.c (check_for_nested_with_variably_modified): Fix typo.


Added:
trunk/gcc/testsuite/gcc.dg/nested-func-6.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-nested.c


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36367



[Bug c/36367] warning for questionable compound expression

2010-04-01 Thread pzhao at gcc dot gnu dot org


--- Comment #4 from pzhao at gcc dot gnu dot org  2010-04-01 09:19 ---
Sorry! I have misunderstood this issue before.
Is it to ask for warn at the following test case?

=
int foo (int a)
{
  return a;
}
int bar (int c)
{
  if (foo (1), c) // Warn foo(1) don't have obvious side-effects here?
return 1;
  else
return 0;
}
===


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

   Last reconfirmed|2008-05-29 12:00:46 |2010-04-01 09:19:19
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36367



[Bug c/36774] -Wmissing-prototypes triggers on nested functions

2010-04-01 Thread pzhao at gcc dot gnu dot org


--- Comment #4 from pzhao at gcc dot gnu dot org  2010-04-02 02:53 ---
Testing patch


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2008-12-27 22:12:20 |2010-04-02 02:53:34
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36774



[Bug c/36774] -Wmissing-prototypes triggers on nested functions

2010-04-05 Thread pzhao at gcc dot gnu dot org


--- Comment #5 from pzhao at gcc dot gnu dot org  2010-04-05 13:44 ---
Proposed patch
http://gcc.gnu.org/ml/gcc-patches/2010-04/msg00078.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36774



[Bug c/36774] -Wmissing-prototypes triggers on nested functions

2010-04-11 Thread pzhao at gcc dot gnu dot org


--- Comment #6 from pzhao at gcc dot gnu dot org  2010-04-12 03:43 ---
Subject: Bug 36774

Author: pzhao
Date: Mon Apr 12 03:43:21 2010
New Revision: 158214

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158214
Log:
gcc/
2010-04-12  Shujing Zhao  

PR c/36774
* c-decl.c (start_function): Move forward check for nested function.

gcc/testsuite/
2010-04-12  Shujing Zhao  

PR c/36774
* gcc.dg/pr36774-1.c: New test.
* gcc.dg/pr36774-2.c: New test.


Added:
trunk/gcc/testsuite/gcc.dg/pr36774-1.c
trunk/gcc/testsuite/gcc.dg/pr36774-2.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-decl.c
trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36774



[Bug c/36774] -Wmissing-prototypes triggers on nested functions

2010-04-11 Thread pzhao at gcc dot gnu dot org


--- Comment #7 from pzhao at gcc dot gnu dot org  2010-04-12 04:00 ---
Fixed on trunk.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36774



[Bug c/32207] missing warnings about address of 'x'.

2010-04-26 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-04-27 02:40 ---
Subject: Bug 32207

Author: pzhao
Date: Tue Apr 27 02:40:19 2010
New Revision: 158765

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158765
Log:
gcc/
2010-04-27  Shujing Zhao  

PR c/32207
* c-typeck.c (build_binary_op): Move forward check for comparison
pointer with null pointer constant and adjust the diagnostic message.

gcc/testsuite/
2010-04-27  Shujing Zhao  

PR c/32207
* gcc.dg/pr32207.c: New test.
* gcc.dg/misc-column.c: Adjust expected warning.
* gcc.dg/Walways-true-1.c: Likewise.
* gcc.dg/Walways-true-2.c: Likewise.
* gcc.dg/warn-addr-cmp.c: Likewise.


Added:
trunk/gcc/testsuite/gcc.dg/pr32207.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-typeck.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/Walways-true-1.c
trunk/gcc/testsuite/gcc.dg/Walways-true-2.c
trunk/gcc/testsuite/gcc.dg/misc-column.c
trunk/gcc/testsuite/gcc.dg/warn-addr-cmp.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32207



[Bug c/32207] missing warnings about address of 'x'.

2010-04-26 Thread pzhao at gcc dot gnu dot org


--- Comment #4 from pzhao at gcc dot gnu dot org  2010-04-27 02:42 ---
Fix for trunk.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32207



[Bug c++/43906] New: missing warnings the comparison between an address with a null pointer constant

2010-04-26 Thread pzhao at gcc dot gnu dot org
As Manuel mentioned at http://gcc.gnu.org/ml/gcc-patches/2010-04/msg01425.html.
C++ have the same problem as pr32207.

extern void z();
void f() { if ( z ) z(); }
void g() { if ( z != 0 ) z(); }
void h() { if ( z != (void*)0 ) z(); }

t.C: In function ‘void f()’:
t.C:2:19: warning: the address of ‘void z()’ will always evaluate as ‘true’
[-Waddress]
t.C: In function ‘void g()’:
t.C:3:22: warning: the address of ‘void z()’ will never be NULL [-Waddress]


-- 
   Summary: missing warnings the comparison between an address with
a null pointer constant
   Product: gcc
   Version: 4.6.0
Status: UNCONFIRMED
  Keywords: diagnostic
  Severity: normal
  Priority: P3
 Component: c++
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: pzhao at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43906



[Bug c++/43779] Parts of message not available for translation

2010-04-28 Thread pzhao at gcc dot gnu dot org


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-04-18 18:24:42 |2010-04-28 09:23:04
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43779



[Bug c/40885] build_indirect_ref i18n problems

2009-12-16 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2009-12-17 03:22 ---
Subject: Bug 40885

Author: pzhao
Date: Thu Dec 17 03:22:22 2009
New Revision: 155302

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155302
Log:
2009-12-17  Shujing Zhao  

PR c/40885
* c-common.h (ref_operator): New type.
(build_indirect_ref): Adjust prototype with new argument.
* c-typeck.c (build_indirect_ref): Accept ref_operator as argument and
emit the diagnostics for easy translation.
(build_array_ref): Update calls to build_indirect_ref.
* c-omp.c (c_finish_omp_atomic): Likewise.
* c-parser.c (c_parser_unary_expression,
c_parser_postfix_expression_after_primary): Likewise.

cp/
2009-12-17  Shujing Zhao  

* typeck.c (build_indirect_ref): Update the argument.
(build_x_indirect_ref): Likewise.
(cp_build_indirect_ref): Update the argument and emit the diagnostics
for easy translation.
(build_class_member_access_expr, build_array_ref,
get_member_function_from_ptrfunc): Update calls.
* cp-tree.h (build_x_indirect_ref, cp_build_indirect_ref): Update
prototypes.
* call.c (build_new_op, convert_like_real, build_x_va_arg,
build_over_call): Update calls.
* class.c (build_base_path, build_simple_base_path, build_vfn_ref):
Likewise.
* decl.c (start_preparsed_function): Likewise.
* except.c (expand_start_catch_block, build_throw): Likewise.
* init.c (emit_mem_initializers, expand_virtual_init,
expand_virtual_init, build_new_1, build_vec_init, build_delete,
build_vec_delete): Likewise.
* parser.c (cp_parser_unary_expression): Likewise.
* pt.c (tsubst_copy_and_build): Likewise.
* rtti.c (build_headof, get_tinfo_decl_dynamic, get_typeid): Likewise.
* semantics.c (finish_non_static_data_member, thisify_lambda_field):
Likewise.
* tree.c (build_dummy_object, stabilize_expr): Likewise.
* typeck2.c (build_x_arrow): Likewise.

testsuite/
2009-12-17  Shujing Zhao  

* g++.old-deja/g++.mike/net31.C: Make expected dg-error strings
explicit.
* g++.old-deja/g++.bugs/900213_02.C: Likewise.
* g++.old-deja/g++.bugs/900215_02.C: Likewise.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-common.h
trunk/gcc/c-omp.c
trunk/gcc/c-parser.c
trunk/gcc/c-typeck.c
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/call.c
trunk/gcc/cp/class.c
trunk/gcc/cp/cp-tree.h
trunk/gcc/cp/decl.c
trunk/gcc/cp/except.c
trunk/gcc/cp/init.c
trunk/gcc/cp/parser.c
trunk/gcc/cp/pt.c
trunk/gcc/cp/rtti.c
trunk/gcc/cp/semantics.c
trunk/gcc/cp/tree.c
trunk/gcc/cp/typeck.c
trunk/gcc/cp/typeck2.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.old-deja/g++.bugs/900213_02.C
trunk/gcc/testsuite/g++.old-deja/g++.bugs/900215_02.C
trunk/gcc/testsuite/g++.old-deja/g++.mike/net31.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40885



[Bug c/40885] build_indirect_ref i18n problems

2009-12-16 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2009-12-17 03:31 ---
Fixed for 4.5.0.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40885



[Bug c++/31665] %s substituted with "built-in"/"library" can't be properly translated

2009-12-18 Thread pzhao at gcc dot gnu dot org


--- Comment #1 from pzhao at gcc dot gnu dot org  2009-12-18 08:50 ---
Subject: Bug 31665

Author: pzhao
Date: Fri Dec 18 08:50:24 2009
New Revision: 155340

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155340
Log:
cp/
2009-12-16  Shujing Zhao  

PR c++/31665
* decl.c (duplicate_decls, grokdeclarator): Put the
diagnostics in full sentences for easy translation and wrapped into
G_().
* typeck.c (build_x_unary_op): Likewise.

testsuite/
2009-12-16  Shujing Zhao  

* g++.old-deja/g++.brendan/misc6.C: Make expected dg-error strings
explicit.


Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/decl.c
trunk/gcc/cp/typeck.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.old-deja/g++.brendan/misc6.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31665



[Bug translation/42467] exgettext should not remove TABs from option help strings

2010-01-07 Thread pzhao at gcc dot gnu dot org


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2009-12-23 11:58:25 |2010-01-07 08:38:27
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42467



[Bug translation/42469] option help strings not properly using TAB

2010-01-07 Thread pzhao at gcc dot gnu dot org


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2009-12-23 12:08:18 |2010-01-07 08:38:44
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42469



[Bug translation/39521] specs diagnostics translation problems

2010-01-08 Thread pzhao at gcc dot gnu dot org


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-01-08 10:29:20
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39521



[Bug translation/42469] option help strings not properly using TAB

2010-01-10 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-01-11 04:28 ---
Subject: Bug 42469

Author: pzhao
Date: Mon Jan 11 04:28:36 2010
New Revision: 155801

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155801
Log:
gcc/po/
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42467
* exgettext: Keep the text before tab character in the option help
string at *.opt file.

gcc/
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42469
* common.opt (Wframe-larger-than=, fcompare-debug=, fdbg-cnt=,
fira-verbose=, flto-compression-level=, fplugin-arg-): Use tab
character between option name and help text.
* c.opt (imultilib): Likewise.

gcc/fortran
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42469
* lang.opt (fblas-matmul-limit=, finit-character=, finit-integer=,
finit-logical=, finit-real=, fmax-array-constructor=): Use tab
character between option name and help text.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/c.opt
trunk/gcc/common.opt
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/lang.opt
trunk/gcc/po/ChangeLog
trunk/gcc/po/exgettext


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42469



[Bug translation/42467] exgettext should not remove TABs from option help strings

2010-01-10 Thread pzhao at gcc dot gnu dot org


--- Comment #5 from pzhao at gcc dot gnu dot org  2010-01-11 04:28 ---
Subject: Bug 42467

Author: pzhao
Date: Mon Jan 11 04:28:36 2010
New Revision: 155801

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155801
Log:
gcc/po/
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42467
* exgettext: Keep the text before tab character in the option help
string at *.opt file.

gcc/
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42469
* common.opt (Wframe-larger-than=, fcompare-debug=, fdbg-cnt=,
fira-verbose=, flto-compression-level=, fplugin-arg-): Use tab
character between option name and help text.
* c.opt (imultilib): Likewise.

gcc/fortran
2010-01-11  Joseph Myers  
Shujing Zhao  

PR translation/42469
* lang.opt (fblas-matmul-limit=, finit-character=, finit-integer=,
finit-logical=, finit-real=, fmax-array-constructor=): Use tab
character between option name and help text.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/c.opt
trunk/gcc/common.opt
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/lang.opt
trunk/gcc/po/ChangeLog
trunk/gcc/po/exgettext


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42467



[Bug translation/42467] exgettext should not remove TABs from option help strings

2010-01-11 Thread pzhao at gcc dot gnu dot org


--- Comment #6 from pzhao at gcc dot gnu dot org  2010-01-11 09:33 ---
Fixed for 4.5.0


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42467



[Bug translation/42469] option help strings not properly using TAB

2010-01-11 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-01-11 09:34 ---
Fixed for 4.5.0


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42469



[Bug translation/39521] specs diagnostics translation problems

2010-01-13 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-01-14 06:28 ---
Subject: Bug 39521

Author: pzhao
Date: Thu Jan 14 06:28:27 2010
New Revision: 155878

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155878
Log:
po/
2010-01-14  Shujing Zhao  

PR translation/39521
* exgettext: Extracted all specs %n strings and the %e strings that %e
is at the start of a line.

2010-01-14  Shujing Zhao  

PR translation/39521
* gcc.c (do_spec_1): Wrapped the error and notice messages of specs
strings with _().


Modified:
trunk/gcc/ChangeLog
trunk/gcc/gcc.c
trunk/gcc/po/ChangeLog
trunk/gcc/po/exgettext


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39521



[Bug translation/39521] specs diagnostics translation problems

2010-01-13 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-01-14 06:36 ---
Fixed for 4.5.0.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39521



[Bug c/40563] -Wc++-compat does not warn about uninitialized const field in struct

2010-07-20 Thread pzhao at gcc dot gnu dot org


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |pzhao at gcc dot gnu dot org
   |dot org |
 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-07-21 02:47:47
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40563



[Bug c/40563] -Wc++-compat does not warn about uninitialized const field in struct

2010-08-17 Thread pzhao at gcc dot gnu dot org


--- Comment #1 from pzhao at gcc dot gnu dot org  2010-08-17 08:25 ---
Subject: Bug 40563

Author: pzhao
Date: Tue Aug 17 08:25:20 2010
New Revision: 163296

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163296
Log:
/gcc
2010-08-17  Shujing Zhao  

PR c/40563
* c-decl.c (diagnose_uninitialized_cst_member): New function.
(finish_decl): Use it to issue a -Wc++-compat warning about
uninitialized const field in struct or union.

(finish_struct): Use strip_array_types.

/gcc/testsuite
2010-08-17  Shujing Zhao  

PR c/40563
* gcc.dg/Wcxx-compat-20.c: New test.


Added:
trunk/gcc/testsuite/gcc.dg/Wcxx-compat-20.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-decl.c
trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40563



[Bug c/40563] -Wc++-compat does not warn about uninitialized const field in struct

2010-08-17 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-08-17 08:28 ---
Fix on trunk


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

   Target Milestone|--- |4.6.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40563



[Bug c/40563] -Wc++-compat does not warn about uninitialized const field in struct

2010-08-17 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-08-17 08:30 ---
Fix on trunk


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40563



[Bug c++/43779] Parts of message not available for translation

2010-04-29 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-04-30 06:16 ---
Subject: Bug 43779

Author: pzhao
Date: Fri Apr 30 06:16:26 2010
New Revision: 158919

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158919
Log:
2010-04-30  Shujing Zhao  

PR c++/43779
* typeck.c (warn_args_num): New function.
(convert_arguments): Use warn_args_num to print the diagnostic
messages.

Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/typeck.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43779



[Bug c++/43779] Parts of message not available for translation

2010-05-05 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-05-05 14:18 ---
fixed for 4.6.0


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43779



[Bug c++/30566] -Wshadow warns about clashes between nested function parameters in C++

2010-05-13 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-05-14 03:19 ---
Subject: Bug 30566

Author: pzhao
Date: Fri May 14 03:19:32 2010
New Revision: 159383

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=159383
Log:
gcc/cp/
2010-05-14  Shujing Zhao  

PR c++/30566
* name-lookup.c (pushdecl_maybe_friend): Avoid the warnings about
shadowing the outer parameter or variables by the declaration of
nested function in nested structure or class. Warn the shadowing by
the declaration of nested lambda expression.

gcc/testsuite/
2010-05-14  Shujing Zhao  

PR c++/30566
* testsuite/g++.dg/warn/Wshadow-4.C: Adjust.
* testsuite/g++.dg/warn/Wshadow-5.C: New test.
* testsuite/g++.dg/warn/Wshadow-6.C: New test.

Added:
trunk/gcc/testsuite/g++.dg/warn/Wshadow-5.C
trunk/gcc/testsuite/g++.dg/warn/Wshadow-6.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/name-lookup.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/warn/Wshadow-4.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30566



[Bug c++/18249] C++ parser i18n problems

2010-05-24 Thread pzhao at gcc dot gnu dot org


--- Comment #1 from pzhao at gcc dot gnu dot org  2010-05-25 06:26 ---
Subject: Bug 18249

Author: pzhao
Date: Tue May 25 06:26:07 2010
New Revision: 159808

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=159808
Log:
2010-05-25  Shujing Zhao  

PR c++/18249
* parser.c: Remove inclusion of dyn-string.h.
(non_integral_constant): New enum.
(name_lookup_error): New enum.
(required_token): New enum.
(cp_parser_required_error): New function.
(cp_parser_require): Change the type of variable token_desc to
required_token and use cp_parser_required_error.
(cp_parser_require_keyword): Likewise.
(cp_parser_error): Use gmsgid as parameter.
(cp_parser_name_lookup_error): Change the type of variable desired to
name_lookup_error and put the diagnostic in the full sentences. Change
caller.
(cp_parser_non_integral_constant_expression): Change the type of the
variable thing to non_integral_constant and put the diagnostics in
full sentences. Change caller.

Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/parser.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18249



[Bug c++/18249] C++ parser i18n problems

2010-05-24 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-05-25 06:57 ---
Fixed for 4.6.0.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18249



[Bug c++/18249] C++ parser i18n problems

2010-05-24 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-05-25 06:57 ---
Fixed.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18249



[Bug c++/38468] Bad order of error messages for function call with wrong number of arguments

2010-06-01 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-06-02 06:53 ---


*** This bug has been marked as a duplicate of 43126 ***


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||DUPLICATE


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38468



[Bug c++/43126] "at this point in file" warnings are upside down

2010-06-01 Thread pzhao at gcc dot gnu dot org


--- Comment #9 from pzhao at gcc dot gnu dot org  2010-06-02 06:53 ---
*** Bug 38468 has been marked as a duplicate of this bug. ***


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||ian at airs dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43126



[Bug c/37724] "initialization from incompatible pointer type" does not say which field is being initialized

2010-06-07 Thread pzhao at gcc dot gnu dot org


--- Comment #8 from pzhao at gcc dot gnu dot org  2010-06-08 03:56 ---
Subject: Bug 37724

Author: pzhao
Date: Tue Jun  8 03:56:40 2010
New Revision: 160418

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160418
Log:
gcc/
2010-06-08  Andrew Pinski 
Shujing Zhao  

PR c/37724
* c-typeck.c (convert_for_assignment): Call pedwarn_init if the
implicit bad conversions is initialization.
(error_init): Use gmsgid instead of msgid for argument name and change
the call for error.
(pedwarn_init): Use gmsgid instead of msgid for argument name and
change the call for pedwarn.
(warning_init): Use gmsgid instead of msgid for argument name and
change the call for warning.

gcc/testsuite/
2010-06-08  Andrew Pinski 
Shujing Zhao  

PR c/37724
* gcc.dg/c90-const-expr-10.c: Adjust.
* gcc.dg/c99-const-expr-10.c: Adjust.
* gcc.dg/init-bad-7.c: New.


Added:
trunk/gcc/testsuite/gcc.dg/init-bad-7.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-typeck.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/c90-const-expr-10.c
trunk/gcc/testsuite/gcc.dg/c99-const-expr-10.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37724



[Bug c/37724] "initialization from incompatible pointer type" does not say which field is being initialized

2010-06-07 Thread pzhao at gcc dot gnu dot org


--- Comment #9 from pzhao at gcc dot gnu dot org  2010-06-08 04:02 ---
Fixed for trunk.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37724



[Bug c/44517] improve diagnostic for mispelled typename in function declaration

2010-06-18 Thread pzhao at gcc dot gnu dot org


--- Comment #1 from pzhao at gcc dot gnu dot org  2010-06-18 10:18 ---
Confirmed. The diagnostic of clang is better.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||pzhao at gcc dot gnu dot org
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-06-18 10:18:00
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44517



[Bug c/44517] improve diagnostic for mispelled typename in function declaration

2010-06-25 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-06-25 07:32 ---
Subject: Bug 44517

Author: pzhao
Date: Fri Jun 25 07:31:49 2010
New Revision: 161363

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161363
Log:
gcc/
2010-06-25  Shujing Zhao  

PR c/44517
* c-parser.c (c_parser_parms_list_declarator): Return NULL if one of
parameters are not good.
(c_parser_parameter_declaration): Error unknown type name if the type
name can't start declaration specifiers.

gcc/testsuite/
2010-06-25  Shujing Zhao  

PR c/44517
* gcc.dg/noncompile/pr44517.c: New.
* gcc.dg/noncompile/990416-1.c: Adjust expected error.


Added:
trunk/gcc/testsuite/gcc.dg/noncompile/pr44517.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/c-parser.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/noncompile/990416-1.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44517



[Bug c/44519] improve message for missing ";" after struct

2010-06-29 Thread pzhao at gcc dot gnu dot org


--- Comment #1 from pzhao at gcc dot gnu dot org  2010-06-29 08:34 ---
Confirmed.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-06-29 08:34:37
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44519



[Bug c/44519] improve message for missing ";" after struct

2010-06-29 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-06-30 03:55 ---
C++ has the same problem.

Testcase:
class foo { int x; }

typedef int bar;

g++-4.6:
a.C:3:13: error: two or more data types in declaration of ‘bar’

clang++:
a.C:1:21: error: expected ';' after class
class foo { int x; }
^
    ;


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 CC|        |pzhao at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44519



[Bug c++/22138] Better error message for rejecting local template declaration.

2010-07-04 Thread pzhao at gcc dot gnu dot org


--- Comment #2 from pzhao at gcc dot gnu dot org  2010-07-05 05:36 ---
Subject: Bug 22138

Author: pzhao
Date: Mon Jul  5 05:36:02 2010
New Revision: 161818

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161818
Log:
gcc/cp/
2010-07-05  Shujing Zhao  

PR c++/22138
* parser.c (cp_parser_primary_expression): Error if local template is
declared.

gcc/testsuite/
2010-07-05  Shujing Zhao  

PR c++/22138
* g++.dg/parse/template25.C: New.


Added:
trunk/gcc/testsuite/g++.dg/parse/template25.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/parser.c
trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22138



[Bug c++/22138] Better error message for rejecting local template declaration.

2010-07-04 Thread pzhao at gcc dot gnu dot org


--- Comment #3 from pzhao at gcc dot gnu dot org  2010-07-05 06:13 ---
Fixed for trunk.


-- 

pzhao at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22138



[Bug c/44854] New: Improve diagnostic for missing member name or ';' in a struct

2010-07-07 Thread pzhao at gcc dot gnu dot org
Testcase:
struct foo { int };

gcc-4.6:
test.c:1:18: error: expected identifier or ‘(’ before ‘}’ token
test.c:1:18: error: expected specifier-qualifier-list at end of input

clang:
test.c:1:18: error: expected member name or ';' after declaration specifiers
struct foo { int };
 ~~~ ^

Clang knows what you mean and the diagnostic message is better.


-- 
   Summary: Improve diagnostic for missing member name or ';' in a
struct
   Product: gcc
   Version: 4.6.0
Status: UNCONFIRMED
  Keywords: diagnostic
  Severity: normal
  Priority: P3
 Component: c
AssignedTo: unassigned at gcc dot gnu dot org
    ReportedBy: pzhao at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44854