[Webkit-unassigned] [Bug 30710] [Qt] QWebHistory::saveState() is inconsistent with the Qt API

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=30710





--- Comment #14 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2009-10-28 11:02:55 PDT ---
(In reply to comment #13)
> (In reply to comment #12)
> > (From update of attachment 42027 [details] [details])
> > QDataStream's version is for the internal format of QDataStream. We should
> > follow the example in the docs:
> > 
> > http://doc.trolltech.com/4.5/qdatastream.html
> > 
> > Ie, a payload-spesific version where we try to load whatever we can from
> > earlier versions, etc.
> 
> We should keep the versioning internal, and always save in the latest format.
> Reading will read only if the version is supported.
> 
> This is more or less what me and Tor Arne agreed upon.
> 
> If needed in the future, it will still be possible to add support for saving in
> older formats.

The above was implemented and landed in 50224.

The only thing missing is making the internal QDataStream/QByteArray in the
stream operators go away.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.