----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108731/#review30130 -----------------------------------------------------------
This review has been submitted with commit cb8905ffb4b67d148add75309bb00094ec9c711b by Bernd Buschinski to branch master. - Commit Hook On Feb. 4, 2013, 12:02 a.m., Bernd Buschinski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108731/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2013, 12:02 a.m.) > > > Review request for kdelibs. > > > Description > ------- > > khtml/canvas: Use extra bool to check if PainterPath is empty, > storring a Inf/Inf member no longer work with recent Qt versions and > storring NaN/NaN never worked, Qt just ignores storring them. > > And set Starting point in arcTo if Path is empty, according to: > http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.path.arcTo.empty > http://philip.html5.org/tests/canvas/suite/tests/spec.html#ensure-there-is-a-subpath > and test: > http://philip.html5.org/tests/canvas/suite/tests/2d.path.arcTo.ensuresubpath.1.html > > > Diffs > ----- > > khtml/html/html_canvasimpl.h f31954f > khtml/html/html_canvasimpl.cpp 93d5db7 > > Diff: http://git.reviewboard.kde.org/r/108731/diff/ > > > Testing > ------- > > > Thanks, > > Bernd Buschinski > >