Closed
Bug 30346
Opened 25 years ago
Closed 25 years ago
stop building mozilla/widget/timer/src/windows/nsTimer.cpp on the mac
Categories
(SeaMonkey :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: sspitzer, Assigned: michael.j.lowe)
Details
looking at
http://tinderbox.mozilla.org/SeaMonkey/warn952120889.5858.html#ramiro, I see
that we are getting warning when building
mozilla/widget/timer/src/windows/nsTimer.cpp on the mac.
the problem is, we shouldn't be building that file on the mac, right?
Reporter | ||
Comment 1•25 years ago
|
||
<smfr> oh, i see what you mean. not the warning itself, but the fact that we are
building the windows code
<smfr> y'know, i suspect that lxr is messing up
this might not be michaels bug. it might be a problem with how we generate the
warnings log file.
from sfraser:
"we're prolly just doing an lxr search on the filename to generate warnigns, and
it finds the windows file first. i'm pretty sure the mac project builds the
correct file. so i wonder if that warning makes sense for the mac file"
slamm, is this your bug?
Assignee | ||
Comment 2•25 years ago
|
||
The problem is in:
http://lxr.mozilla.org/seamonkey/source/widget/timer/src/mac/nsTimer.cpp
The methods GetPriority & GetType should be returning values. See below:
92 virtual PRUint32 GetPriority() {};
93 virtual void SetPriority(PRUint32 aPriority) {};
94
95 virtual PRUint32 GetType() {};
96 virtual void SetType(PRUint32 aType) {};
Comment 3•25 years ago
|
||
These methods are not called, because repeating timers are unimplemented on Mac
(see bug 904). I'm working on that.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
Comment 4•25 years ago
|
||
can one of you all verify this. I'm not sure I'm teh right person to make that
call. thanks
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•