Differences between revisions 10 and 11
Revision 10 as of 2014-10-13 10:23:32
Size: 1341
Comment: BUG:PR60664 - bool / out of range int comparison warning failure (easy) fixed
Revision 11 as of 2014-10-15 10:11:27
Size: 2655
Comment: add trivial patch
Deletions are marked like this. Additions are marked like this.
Line 24: Line 24:

=== A patch exists but needs to be tested ===

 * Trivial patch:

   {{{#!highlight diff numbers=disable
Index: c-decl.c
===================================================================
--- c-decl.c (revision 216164)
+++ c-decl.c (working copy)
@@ -6754,7 +6754,6 @@
   tree types = 0;
   tree others = 0;
 
- static bool explained_incomplete_types = false;
   bool gave_void_only_once_err = false;
 
   arg_info->had_vla_unspec = current_scope->had_vla_unspec;
@@ -6857,19 +6856,12 @@
      {
        if (b->id)
   /* The %s will be one of 'struct', 'union', or 'enum'. */
- warning (0, "%<%s %E%> declared inside parameter list",
+ warning (0, "%<%s %E%> declared inside parameter list will not be visible outside of this function",
     keyword, b->id);
        else
   /* The %s will be one of 'struct', 'union', or 'enum'. */
- warning (0, "anonymous %s declared inside parameter list",
+ warning (0, "anonymous %s declared inside parameter list will not be visible outside of this function",
     keyword);
-
- if (!explained_incomplete_types)
- {
- warning (0, "its scope is only this definition or declaration,"
- " which is probably not what you want");
- explained_incomplete_types = true;
- }
      }
 
    tag.id = b->id;
}}}

These bugs (also called Problem Reports) are great starting points for new contributors to GCC. If you have doubts or questions, don't be afraid to ask in the bugzilla page of each bug.

C-family (bug affects both C and C++)

A patch exists but needs to be tested

Analyzed what needs to be fixed, but no patch

C only

A patch exists but needs to be tested

  • Trivial patch:
    • Index: c-decl.c
      ===================================================================
      --- c-decl.c    (revision 216164)
      +++ c-decl.c    (working copy)
      @@ -6754,7 +6754,6 @@
         tree types    = 0;
         tree others   = 0;
       
      -  static bool explained_incomplete_types = false;
         bool gave_void_only_once_err = false;
       
         arg_info->had_vla_unspec = current_scope->had_vla_unspec;
      @@ -6857,19 +6856,12 @@
                  {
                    if (b->id)
                      /* The %s will be one of 'struct', 'union', or 'enum'.  */
      -               warning (0, "%<%s %E%> declared inside parameter list",
      +               warning (0, "%<%s %E%> declared inside parameter list will not be visible outside of this function",
                               keyword, b->id);
                    else
                      /* The %s will be one of 'struct', 'union', or 'enum'.  */
      -               warning (0, "anonymous %s declared inside parameter list",
      +               warning (0, "anonymous %s declared inside parameter list will not be visible outside of this function",
                               keyword);
      -
      -             if (!explained_incomplete_types)
      -               {
      -                 warning (0, "its scope is only this definition or declaration,"
      -                          " which is probably not what you want");
      -                 explained_incomplete_types = true;
      -               }
                  }
       
                tag.id = b->id;
      

Analyzed what needs to be fixed, but no patch

  • PR23087 Misleading warning, "... differ in signedness"

C++ only

A patch exists but needs to be tested

A patch exists but needs to be updated

Analyzed what needs to be fixed, but no patch

Fortran

Analyzed what needs to be fixed, but no patch

None: EasyHacks (last edited 2017-09-13 11:36:40 by NathanSidwell)