[Bug 752123] harness: don't run code inside g_assert()

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=752123

Luis de Bethencourt <luis at debethencourt.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #307072|rejected                    |committed
             status|                            |

--- Comment #9 from Luis de Bethencourt <luis at debethencourt.com> ---
Review of attachment 307072:
 --> (https://bugzilla.gnome.org/review?bug=752123&attachment=307072)

After some discussion in IRC. It was decided to push this change.

commit 60de1f26c78feb0cde6d3f82cf86cf35daa71cc0
Author: Luis de Bethencourt <luis at debethencourt.com>
Date:   Wed Aug 5 14:05:25 2015 +0100

    harness: don't run code inside g_assert

    Even though asserts can't be disabled in GstHarness, Coverity still
    complains about running code inside them. Moving the code to outside the
    g_asserts().

    CID #1311326, #1311327, #1311328

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.