This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] add return values to return statements in non-void functions


On Wed, Apr 23, 2014 at 5:23 PM, Doug Evans <dje@google.com> wrote:
> David Blaikie writes:
>  > These missing return values cause build breaks with clang which
>  > defaults the -Wreturn-type warning to an error in this case.
>  >
>  > Add the right return values so these tests can run when using clang.
>  > commit ce1534e51863af5d935cdc63c44df0fa64a46653
>  > Author: David Blaikie <dblaikie@gmail.com>
>  > Date:   Fri Apr 11 17:20:49 2014 -0700
>  >
>  >     Add return value for non-void function return statements to fix error in clang build.
>  >
>  >     Clang defaults this warning to an error, breaking the build & causing
>  >     these tests not to run.
>  >
>  >     gdb/testsuite/
>  >
>  >      * gdb.mi/non-stop.c: Add return value for non-void function return
>  >      statement.
>  >      * gdb.threads/staticthreads.c: Ditto.
>
> ChangeLog rules require being more specific about where the change happened.
> E.g.,
>
>         * gdb.mi/non-stop.c (worker): Add return value for non-void function
>         return statement.
>         * gdb.threads/staticthreads.c (thread_function): Ditto.
>
> Ok with those changes.
> Thanks!

Bleah.  Missed that this one was also already approved.
[that's what I get for switching back and forth between mail readers]


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]