Re: Review Request 127560: Remove U(...) macro.

2016-04-04 Thread Andreas Cord-Landwehr
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127560/ --- (Updated April 4, 2016, 10:57 a.m.) Status -- This change has been m

Re: Review Request 127560: Remove U(...) macro.

2016-04-03 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127560/#review94216 --- Ship it! Hmm, but why isn't kalzium in that lxr output? Is

Re: Review Request 127560: Remove U(...) macro.

2016-04-03 Thread Andreas Cord-Landwehr
> On April 3, 2016, 2:21 nachm., David Faure wrote: > > The rest of frameworks is the wrong place to grep, please grep the > > files/dirs listed in > > https://lxr.kde.org/search?_filestring=&_string=KUnitConversion Everything looks fine in the meaning that the macro is nowhere used in the fi

Re: Review Request 127560: Remove U(...) macro.

2016-04-03 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127560/#review94214 --- The rest of frameworks is the wrong place to grep, please gre

Review Request 127560: Remove U(...) macro.

2016-04-03 Thread Andreas Cord-Landwehr
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127560/ --- Review request for KDE Frameworks and John Layt. Repository: kunitconvers