On 03.06.2019, at 00:37, Andreas Rheinhardt <andreas.rheinha...@gmail.com> wrote:
> Up until now, a temporary variable was used and initialized every time a > value was read in CBS; if reading turned out to be successfull, this > value was overwritten (without having ever been looked at) with the > value read if reading was successfull; on failure the variable wasn't > touched either. Therefore these initializations can be and have been > removed. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > And? What did the ancient compilers say? Not sure how to read that, but some compilers probably produce "may be used uninitialized" warnings after this change. IMHO it would be better to need evidence of an advantage to remove variable initialization for non-trivial code, even if they are unnecessary. Your commit message doesn't seem to mention one at least. But I am happy to let some maintainer have the last word. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
