[FFmpeg-devel] [PATCH] lavu/hwcontext_vaapi: ignore nonexistent device in default DRM device selection

Anton Khirnov anton at khirnov.net
Mon Nov 27 10:37:38 EET 2023


Quoting Xiang, Haihao (2023-11-27 05:42:20)
> > 
> > Also, the else is unnecessary since the if() block ends with continue.
> > 
> 
> I wanted to print a message if all devices don't exist. I'll remove it in the
> new version if you think such info is unnecessary. 

I don't object to the message, I mean just that

if (foo) {
  ....
  continue;
} else
  bar;

is equivalent to

if (foo) {
  ....
  continue;
}

bar;

The latter saves you one level of indentation and I find it more
readable.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list