On Thu, Apr 16, 2020 at 1:01 AM Sean Young <sean@xxxxxxxx> wrote: > > On Wed, Apr 15, 2020 at 04:34:08PM -0700, Rosen Penev wrote: > > On Fri, Apr 10, 2020 at 5:28 PM Rosen Penev <rosenp@xxxxxxxxx> wrote: > > > > > > On Wed, Apr 8, 2020 at 1:42 PM Ariel D'Alessandro > > > <ariel@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > > > > > Currently supports building libraries and tools found in lib/ and > > > > utils/ directories. > > > Excellent. This will make it easier to run the code against static > > > code analyzers. > > This is worse than I thought. v4l-utils fails to compile with clang > > (and by extension clang-tidy as it uses nested functions. > > That's down to some code I wrote. I was hoping clang was going to add > nested function support at some point, but they haven't. > > I've just sent a patch to the list removing the nested functions from > v4l-utils. I don't think the code is improved by this, but I think being > able to build with clang takes precendence. I agree. Doesn't seem even C++ allows them (lambdas are different). > > > Sean
