Review Request 128909: initial, minimal support for OS X

Thiago Macieira thiago at kde.org
Thu Sep 15 05:22:48 BST 2016


On quarta-feira, 14 de setembro de 2016 17:58:26 PDT René J.V. Bertin wrote:
> > On Sept. 14, 2016, 7:33 p.m., Anthony Fieroni wrote:
> > > processcore/processes_darwin_p.cpp, line 104
> > > <https://git.reviewboard.kde.org/r/128909/diff/1/?file=476875#file476875
> > > line104>> > 
> > >     *inline* must be used in function definition not in declaration.
> 
> I didn't see the point of the way the keyword is being used here either, but
> I aligned myself to the current practice.
> 
> So I take it I shouldn't only correct `argMax()`?

It's all in the same .cpp, it doesn't matter. Inline is correct where it is.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   Software Architect - Intel Open Source Technology Center





More information about the kde-core-devel mailing list