On 28 sept. 2014, at 03:26, Thiago Macieira thiago.macie...@intel.com wrote:
On Sunday 28 September 2014 01:02:11 Samuel Gaist wrote:
Hi,
Following a post on the forum, I've checked and there's been a behavior
change in QUrl's setPath between Qt 4 and Qt 5 that is not mentioned in the
C++ API changes chapter.
If I understood correctly:
QUrl example1(http://www.example.com;);
example1.setPath(pub/something);
makes example1 invalid in Qt 5 due to the fact that pub/something is a
relative path (following QUrl documentation and test) but in Qt 4 the
result is http://www.example.com/pub/something;.
Should it be considered bug in Qt 4 that needs fixing ? However fixing it
might break existing application that could be relying on that behavior. In
this case, simply add the API break in Qt 5's documentation ?
Yes, it's a bug in Qt 4, bug I won't fix it because it's not that important
and
would require a major change.
QUrl in Qt 4 has quite a few known issues with encoding and decoding of
delimiters too. And its QString constructor is a completely flawed design and
should never be used.
QUrl changed considerably in Qt 5 to comply better with the URL
specifications
and with brokenness out there. If we add anything to the documentation, it
would be the previous sentence, with no extra details.
I remember now following a discussion about that matter some time ago.
Fine for me. I'll update the API change doc to include that so future users
won't be surprised.
Samuel
___
Development mailing list
Development@qt-project.org
http://lists.qt-project.org/mailman/listinfo/development