RESOLVED FIXED 3828
GW: add QTime::addMSecs() suppport and currentTime() support to KWQ 
https://bugs.webkit.org/show_bug.cgi?id=3828
Summary GW: add QTime::addMSecs() suppport and currentTime() support to KWQ
Eric Seidel (no email)
Reported 2005-07-02 20:40:14 PDT
Simple patch, adds QTime::addMSecs() suppport and currentTime() support to KWQ
Attachments
self explanitory patch. (1.29 KB, patch)
2005-07-02 20:40 PDT, Eric Seidel (no email) 		mjs: review-
DetailsFormatted DiffDiff
Fixes mjs's concern (1.25 KB, patch)
2005-07-04 01:10 PDT, Eric Seidel (no email) 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2005-07-02 20:40:47 PDT
Created attachment 2767 [details] self explanitory patch.
Maciej Stachowiak
Comment 2 2005-07-03 03:02:53 PDT
Comment on attachment 2767 [details] self explanitory patch. This appears buggy. It seems to add msecs to the default time rather than the current time. One way around this would be to copy-construct QTime from *this, and since the default time value is 0 this is quite likely wrong.
Maciej Stachowiak
Comment 3 2005-07-03 03:03:30 PDT
Oops, my comment got a little garbled, hope it still makes sense.
Eric Seidel (no email)
Comment 4 2005-07-04 01:10:19 PDT
Created attachment 2781 [details] Fixes mjs's concern
Note You need to log in before you can comment on or make changes to this bug.