[Poppler-bugs] [Bug 100552] test-render-to-file.cpp

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=100552

--- Comment #8 from Albert Astals Cid <aacid at kde.org> ---
(In reply to Gwenaël Cléon from comment #6)
> Could you please be a little bit less agressive !
> 
> For example in the header poppler-qt5.h we can see :
> 
> #include <QtCore/QDateTime>
> 
> and the Qt documentation *clearly* states that the header for QDateTime is
> #include <QDateTime>

I can change it if it makes you happier.

> 
> So can you please explain that to me, as I am just a fool and you understood
> everything ?

Maybe?

> http://doc.qt.io/qt-5/search-results.html?q=QDateTime

Once upon a time Qt recommended to use the Module/Class include, then they
realized it was a stupid idea because it mean if they changed Class from
ModuleA to ModuleB things would stop compiling so stopped recommending the
Module/Class include in favour to just Class include. But both are correct and
work, if it doesn't work it means the build system is wrongly set and it needs
to be fixed there, but people keep sending the wrong patch, and then they
complain because i tell them the patch is wrong.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/poppler-bugs/attachments/20170911/8ff8664b/attachment.html>