On Sat, Jun 29, 2013 at 01:03:49PM +0200, Luca Barbato wrote: > On 06/29/2013 12:15 PM, Diego Biurrun wrote: > > On Sat, Jun 29, 2013 at 06:17:37AM +0200, Luca Barbato wrote: > >> --- > >> libavcodec/mjpegdec.c | 268 > >> +++++++++++++++++++++++++------------------------- > >> 1 file changed, 134 insertions(+), 134 deletions(-) > > > > The log message is rather nondescript. I cannot easily see what you > > are trying to achieve here.. > > Remove an indentation level.
The patch is OK, the log message should be more explanatory. Maybe: mjpegdec: cosmetics: Move code out of else case to simplify control flow Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
