> On Sept. 27, 2013, 10:10 a.m., Sebastian Kügler wrote: > > kdeui/widgets/kbuttongroup.cpp, line 72 > > <http://git.reviewboard.kde.org/r/112848/diff/1/?file=190963#file190963line72> > > > > If you're doing formatting changes, please submit them in a separate > > patch. > > > > It's not so much of an issue here, but makes it easier to review.
Ok. I'll adopt it on next submits - Bruno ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112848/#review40908 ----------------------------------------------------------- On Oct. 4, 2013, 2:37 p.m., Bruno Farias wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112848/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2013, 2:37 p.m.) > > > Review request for kdelibs. > > > Repository: kdelibs > > > Description > ------- > > - Adding new test for kbuttongroup > - Fixing kbuttongroup.cpp identation > > > Diffs > ----- > > kdeui/tests/kbuttongrouptest.h cd9130a9b0840160bdce5a1841735e2050d10a7d > kdeui/tests/kbuttongrouptest.cpp d5f9f88b839d4be88cac81c1e7d0444422dd17ea > kdeui/widgets/kbuttongroup.h de813ea95dca1247698537a966abe474fa9650cf > kdeui/widgets/kbuttongroup.cpp 4057cc023107cd76cc34590f4b1dd91e01717d3d > > Diff: http://git.reviewboard.kde.org/r/112848/diff/ > > > Testing > ------- > > All tests of KButtonGroupTest were passed > > > Thanks, > > Bruno Farias > >
