subject:"\[FFmpeg\-devel\] \[PATCH\] libavformat\: Separate assertions of the form \"av_assertN\(a \&\& b\)\" to \"av_assertN\(a\); av_assertN\(b\)\" for more precise diagnostics."

Re: [FFmpeg-devel] [PATCH] libavformat: Separate assertions of the form "av_assertN(a && b)" to "av_assertN(a); av_assertN(b)" for more precise diagnostics.

2019-05-13 Thread Michael Niedermayer

On Sun, May 12, 2019 at 05:49:00AM -0700, Adam Richter wrote: > This is the first of what I expect to be several patches to convert > assertions of the > form "assert(a && b)" to "assert(a); assert(b)". > > Here are some reasons why I think this would be an improvement. This > lengthy argument

[FFmpeg-devel] [PATCH] libavformat: Separate assertions of the form "av_assertN(a && b)" to "av_assertN(a); av_assertN(b)" for more precise diagnostics.

2019-05-12 Thread Adam Richter

This is the first of what I expect to be several patches to convert assertions of the form "assert(a && b)" to "assert(a); assert(b)". Here are some reasons why I think this would be an improvement. This lengthy argument is not included in the patch attachment. 1. Assertion failures are often