> On March 26, 2011, 6:59 p.m., Thomas Baumgart wrote: > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investprocessing.cpp, > > line 1392 > > <http://svn.reviewboard.kde.org/r/6633/diff/1/?file=45858#file45858line1392> > > > > You could easily use an else branch here to avoid the mid-function > > return and thus moving it to the end of the function. Then you can even > > omit the statement completely. > > Allan Anderson wrote: > Done all those. > > Any further comments, anyone?
Am I OK to proceed, please? - Allan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6633/#review10030 ----------------------------------------------------------- On March 26, 2011, 8:56 p.m., Allan Anderson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6633/ > ----------------------------------------------------------- > > (Updated March 26, 2011, 8:56 p.m.) > > > Review request for kmymoney. > > > Summary > ------- > > Previously, it was necessary to type in the name whenever a security was to > be imported. Instead, > a KComboBox is used, and security names entered are saved in the resource > file, to save inputting > each time. A button has been added to allow removal of any unwanted or > erroneous names from the > selection list. > > Also, a decimal separator now has to be selected before import, to avoid > accidental use of a previous, > wrong type, which could affect imported values. > > > Diffs > ----- > > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlg.h > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlg.cpp > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlgdecl.ui > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvprocessing.cpp > 1226119 > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvutil.cpp > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investmentdlg.cpp > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investprocessing.h > 1226119 > > /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investprocessing.cpp > 1226119 > > Diff: http://svn.reviewboard.kde.org/r/6633/diff > > > Testing > ------- > > Numerous test files used, plus production files. > > Astyle and Krazy2 run. > > > Thanks, > > Allan > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel
