Closed
Bug 1122126
Opened 9 years ago
Closed 9 years ago
add configure check for gmtime_r
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: froydnj, Unassigned)
References
Details
Attachments
(1 file)
1.64 KB,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
We have code that tries to do intelligent things if we have gmtime_r: http://mxr.mozilla.org/mozilla-central/source/js/src/vm/DateTime.cpp#34 but we don't have a configure check for gmtime_r: http://mxr.mozilla.org/mozilla-central/search?string=gmtime_r it's worth noting that we do check for localtime_r: http://mxr.mozilla.org/mozilla-central/source/configure.in#3014 http://mxr.mozilla.org/mozilla-central/source/js/src/configure.in#2437 so I'm not sure how we missed gmtime_r. bug 850300 suggests that we might have checked for gmtime_r once upon a time, but the configure test was busted, or something?
Reporter | ||
Comment 1•9 years ago
|
||
It looks like we use gmtime_r in several places without configure checks, so maybe we can just declare victory and use it everywhere?
Reporter | ||
Comment 2•9 years ago
|
||
I don't know whether we *really* need this; we have unguarded uses of gmtime_r in the tree, but we also have guarded uses of gmtime_r. The Linux man page says that gmtime_r is specified in SUSv2 (!) and it also sounds like it was specified for POSIX Threads as well. I've opted to be safer here; we could try ripping checks out in a different bug?
Attachment #8549770 -
Flags: review?(mshal)
Comment 3•9 years ago
|
||
(In reply to Nathan Froyd [:froydnj] [:nfroyd] from comment #1) > It looks like we use gmtime_r in several places without configure checks, so > maybe we can just declare victory and use it everywhere? Keeping in mind everywhere != Windows, it might be fine? It looks like code that is also used on Windows already has #ifdef on _WIN32 anyway. Though one other oddball is other-licenses/android/res_debug.c, which uses HAVE_TIME_R instead of HAVE_GMTIME_R.
Updated•9 years ago
|
Attachment #8549770 -
Flags: review?(mshal) → review+
https://hg.mozilla.org/mozilla-central/rev/f844dad04fb0
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•