----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108396/#review25420 -----------------------------------------------------------
Ship it! Since all methods are pure virtual this header describes an interface. It is implemented in mymoneystatementreader.h but usage of the interface description is not enforced. I don't know why. Before we remove the file, we should check if it contains documentation that is not part of mymoneystatementreader.h and move it over. Otherwise, I don't see a problem with removing the file. - Thomas Baumgart On Jan. 13, 2013, 7:08 p.m., Łukasz Maszczyński wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108396/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2013, 7:08 p.m.) > > > Review request for KMymoney. > > > Description > ------- > > removed file kmymoney/converter/imymoneyreader.h which is not used anywhere > in the project > > > Diffs > ----- > > kmymoney/converter/imymoneyreader.h > 6c1a61936c795997ae09fe06882e24459fa7e886 > > Diff: http://git.reviewboard.kde.org/r/108396/diff/ > > > Testing > ------- > > compile from clean workspace > > > Thanks, > > Łukasz Maszczyński > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel