795351 - Replace LL_MAXINT, LL_MAXUINT and LL_MININT with INT64_MAX, UINT64_MAX and INT64_MIN

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[(no longer active)]

This is pretty similar to bug 791906, but for 64-bit values.

Isaac, interested in taking this as well? :-)

Comment 1

[(no longer active)]

We can also replace the function versions of these macros (LL_MaxInt, LL_MaxUint and LL_MinInt) with the stdint macros here as well (in a separate patch of course).

Comment 2

[Isaac Aggrey [:isaac]]

Attachment: Replace LL_MAXINT, LL_MININT, LL_MAXUINT with stdint versions

Comment 3

[(no longer active)]

Comment on attachment 666058 [details] [diff] [review]
Replace LL_MAXINT, LL_MININT, LL_MAXUINT with stdint versions

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

Looks great!  Can you please test it on try?

Comment 4

[Isaac Aggrey [:isaac]]

Attachment: Replace LL_MaxInt(), LL_MinInt(), LL_MaxUint() functions with stdint macros

I should've stuck both patches in one try run; first patch's results are looking green but still in progress (https://tbpl.mozilla.org/?tree=Try&rev=1a991fd3a848) and the second patch (this one) just got pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=0083adfdec5f

Comment 5

[(no longer active)]

Comment on attachment 666081 [details] [diff] [review]
Replace LL_MaxInt(), LL_MinInt(), LL_MaxUint() functions with stdint macros

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

You are quick, sir!  :-)

Comment 6

[(no longer active)]

I feel confident enough about these patches.  I'll just push them to inbound, and have cancelled the try pushes.

Comment 7

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bd990a83194e
https://hg.mozilla.org/integration/mozilla-inbound/rev/70ab9f476d86

Comment 8

[Isaac Aggrey [:isaac]]

(In reply to Ehsan Akhgari [:ehsan] from comment #6)
> I feel confident enough about these patches.  I'll just push them to
> inbound, and have cancelled the try pushes.

Great, I was kinda hoping that would happen...I was feeling bad about wasting try server resources.

(In reply to Ehsan Akhgari [:ehsan] from comment #5)
> Comment on attachment 666081 [details] [diff] [review]
> Replace LL_MaxInt(), LL_MinInt(), LL_MaxUint() functions with stdint macros
> 
> Review of attachment 666081 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> You are quick, sir!  :-)

I try! Even with these simpler bugs, if I had known that I'd get such a kick out of contributing I would've started a long time ago!

Comment 9

[Mozilla RelEng Bot]

Try run for 1a991fd3a848 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1a991fd3a848
Results (out of 219 total builds):
    exception: 70
    success: 141
    warnings: 6
    failure: 2
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-1a991fd3a848

Comment 10

[Mozilla RelEng Bot]

Try run for 0083adfdec5f is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=0083adfdec5f
Results (out of 95 total builds):
    exception: 58
    success: 37
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-0083adfdec5f

Comment 11

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/bd990a83194e
https://hg.mozilla.org/mozilla-central/rev/70ab9f476d86

Comment 12

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Isaac, thanks a lot for your work! It's much appreciated.

Comment 13

[Isaac Aggrey [:isaac]]

(In reply to :Ms2ger from comment #12)
> Isaac, thanks a lot for your work! It's much appreciated.

It's really my pleasure. Thank you, Ehsan, and others for creating a welcoming environment. :-)

Comment 14

[(no longer active)]

(In reply to comment #13)
> (In reply to :Ms2ger from comment #12)
> > Isaac, thanks a lot for your work! It's much appreciated.
> 
> It's really my pleasure. Thank you, Ehsan, and others for creating a welcoming
> environment. :-)

My pleasure! :-)