Bug 3828

Summary:

GW: add QTime::addMSecs() suppport and currentTime() support to KWQ

Product:

WebKit

Reporter:

Eric Seidel (no email) <eric>

Component:

New Bugs

Assignee:

Eric Seidel (no email) <eric>

Status:

RESOLVED FIXED

Severity:

Enhancement

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Bug Depends on:

Bug Blocks:

3250

Attachments:

Description	Flags
self explanitory patch.	mjs: review-
Fixes mjs's concern	mjs: review+

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-	Details Formatted Diff Diff
Fixes mjs's concern (1.25 KB, patch) 2005-07-04 01:10 PDT, Eric Seidel (no email)	mjs: review+	Details Formatted Diff Diff
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.