Hi,
this commit [1] added a new overload to QTextStream::readLine. As a result
of that, calling stream.readLine(0) is now ambiguous:
QString readLine(qint64 maxlen = 0);
bool readLine(QString *line, qint64 maxlen = 0);
While I can easily fix this in the caller (Konsole in this case), I'm
wodnering whether this effect was understood at the time the change was
merged. IMHO it's a bit more user-friendly to preserve source compatibility
by removing the default value in the newly added overload.
Should I send such a patch?
Cheers,
Jan
[1] https://codereview.qt-project.org/98606
--
Trojitá, a fast Qt IMAP e-mail client -- http://trojita.flaska.net/
_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development
