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+

Description Eric Seidel (no email) 2005-07-02 20:40:14 PDT

Simple patch, adds QTime::addMSecs() suppport and currentTime() support to KWQ

Comment 1 Eric Seidel (no email) 2005-07-02 20:40:47 PDT

Created attachment 2767 [details]
self explanitory patch.

Comment 2 Maciej Stachowiak 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.

Comment 3 Maciej Stachowiak 2005-07-03 03:03:30 PDT

Oops, my comment got a little garbled, hope it still makes sense.

Comment 4 Eric Seidel (no email) 2005-07-04 01:10:19 PDT

Created attachment 2781 [details]
Fixes mjs's concern