Skip to content
Commit c3259aba authored by Richard Schwarting's avatar Richard Schwarting
Browse files

Applying patch from bug #682711

 Arnel A. Borja [reporter] 2012-08-26 01:25:35 UTC

Created an attachment (id=222446) [details] [review]
Assert patches

Four patches for doing the following (merged into one file):
- Use assert_not_reached instead of `assert (false)'
- Remove unnecessary `assert (true)'
- Catch errors, assert catch block as not reached
- Use assert_not_reached instead of GLib.Test.fail
- Use Test.message instead of GLib.warning/error/message

I based these changes by looking into other GNOME projects, like GTK+ and
gedit. Seems like they don't use hardcoded asserts, instead using
assert_not_reached, and other things I listed above.
parent 51f284b8
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment