https://bugs.kde.org/show_bug.cgi?id=181290
--- Comment #106 from Kevin Kofler <kevin kofler chello at> 2011-07-27 19:21:20 --- > but at least you should fix the FIXMEs in your code. As far as I can tell, the one function with a FIXME in it (FilePrinter::printFiles, as opposed to FilePrinter::printFile which is what is actually being used) is used nowhere in all of Okular. (I searched the entire code for uses of them and didn't find any.) I have no idea why it's there in the first place. If you think I should just remove that unused function, I can do it. But I didn't want to do that without knowing why it's there… > You are also breaking BC of the okular core library which is something we do > not like. It is necessary to make those API changes to fix this problem cleanly. It could probably be hacked around without adding those extra function arguments, but that doesn't strike me as a good long-term solution. > Also did you try as I asked PDF files where different pages have different > orientations? That case can't really be handled any worse than the status quo, but comment #105 now confirms that my patch also works in that case. -- Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. _______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
