796279 - remove last remaining PR_MAX usages from the tree

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Nathan Froyd [:froydnj]]

Bug 785542 didn't remove a couple usages of PR_MAX because converting them to NS_MAX would have required consistent constexpr support across compilers.  But there's no reason that we have to wait that long; manually inlining what NS_MAX would produce should work just fine.

Comment 1

[(no longer active)]

Fair enough!  Do you wanna take this?  :-)

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: patch

Sure, I can take this!

Comment 3

[(no longer active)]

Comment on attachment 667149 [details] [diff] [review]
patch

Review of attachment 667149 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/style/nsCSSProps.cpp
@@ +101,5 @@
>    //     largest sizeof(#aliasname_) before or including that alias.
>  #define CSS_PROP_ALIAS(aliasname_, propid_, aliasmethod_, pref_)              \
>    eMaxCSSAliasNameSizeBefore_##aliasmethod_,                                  \
>    eMaxCSSAliasNameSizeWith_##aliasmethod_ =                                   \
> +    CSS_MAX(sizeof(#aliasname_), eMaxCSSAliasNameSizeBefore_##aliasmethod_) - 1,

Nit: please #undef this when you're done with it.

::: xpcom/glue/nsTArray.h
@@ +1348,5 @@
>    // __attribute__((aligned(foo))).
>    union {
>      char mAutoBuf[sizeof(nsTArrayHeader) + N * sizeof(elem_type)];
> +    mozilla::AlignedElem<(MOZ_ALIGNOF(Header) > MOZ_ALIGNOF(elem_type))
> +                         ? MOZ_ALIGNOF(Header) : MOZ_ALIGNOF(elem_type)> mAlign;

Nit: please a small comment on why we're doing this.

Comment 4

[Nathan Froyd [:froydnj]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fc59bd8d49ba

Comment 5

[(no longer active)]

I had to back this out as part of a weird reftest breakage on 64-bit platforms:

https://hg.mozilla.org/integration/mozilla-inbound/rev/402e2cf6602d

Example failure log: https://tbpl.mozilla.org/php/getParsedLog.php?id=15760281&tree=Mozilla-Inbound

Comment 6

[Nathan Froyd [:froydnj]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a0f4ab91ee37

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/a0f4ab91ee37