> On Feb. 12, 2015, 2:53 p.m., Albert Astals Cid wrote: > > Is your script smart enough to convert > > > > virtual QByteArray data() const; > > > > into > > > > QByteArray data() const Q_DECL_OVERRIDE; > > > > instead of > > > > virtual QByteArray data() const Q_DECL_OVERRIDE;
Nope. It only deals with 'override'. But in this case I could easily do sth along s/virtual// in the *diff*. - Kevin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122542/#review75924 ----------------------------------------------------------- On Feb. 12, 2015, 2:43 p.m., Kevin Funk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122542/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2015, 2:43 p.m.) > > > Review request for KDE Frameworks. > > > Description > ------- > > This is a huge patch (almost 30 000 lines) covering all frameworks. > > This was done using clang-modernize and a few wrapper scripts. > > If you'll give me the okay, I'll push this to the individual frameworks. > > > Diffs > ----- > > > Diff: https://git.reviewboard.kde.org/r/122542/diff/ > > > Testing > ------- > > Compiles fine on my machine. > > > File Attachments > ---------------- > > frameworks-add-override.patch > > https://git.reviewboard.kde.org/media/uploaded/files/2015/02/12/97a4c658-1517-4ab8-88ee-f76c09146393__frameworks-add-override.patch > > > Thanks, > > Kevin Funk > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
