----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130043/#review102934 -----------------------------------------------------------
This is not binary compatible (i.e. it removes a symbol from a publicly installed header). It should have been removed when we moved to KF5, but since it wasn't if there's no other reason that "we don't use it" I would prefer to keep it until we have some other forced binary compatibility break. - Albert Astals Cid On March 22, 2017, 3:20 p.m., Oliver Sander wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/130043/ > ----------------------------------------------------------- > > (Updated March 22, 2017, 3:20 p.m.) > > > Review request for Okular. > > > Repository: okular > > > Description > ------- > > I was about to simplify the method FilePrinter::psPaperSize in the file > fileprinter.cpp. As it turned out, this method is never called anywhere > anyway. So this patch removes it. > > > Diffs > ----- > > core/fileprinter.h a60c2c0f > core/fileprinter.cpp a27529e7 > > Diff: https://git.reviewboard.kde.org/r/130043/diff/ > > > Testing > ------- > > Grepping for the method name doesn't return anything. > > Code still builds without the method. > > > Thanks, > > Oliver Sander > >
