CS fixes to match context of other redbean code #141

Merged
merged 1 commit into from Mar 28, 2012

Conversation

Projects
None yet
2 participants
@damianb
Contributor

damianb commented Mar 24, 2012

Minor changes, does not change functionality.

@damianb

This comment has been minimized.

Show comment Hide comment
@damianb

damianb Mar 24, 2012

Contributor

Question regarding lines 149 and 160 - specifically, this:

    $value < 2147483648

Why not use PHP_INT_MAX instead? Is there a reason for using the 32-bit maximum integer value specifically, even when PHP may be a 64 bit build? (just curious)

Contributor

damianb commented Mar 24, 2012

Question regarding lines 149 and 160 - specifically, this:

    $value < 2147483648

Why not use PHP_INT_MAX instead? Is there a reason for using the 32-bit maximum integer value specifically, even when PHP may be a 64 bit build? (just curious)

gabordemooij pushed a commit that referenced this pull request Mar 28, 2012

Gabor de Mooij
Merge pull request #141 from damianb/patch-3-cs
CS fixes to match context of other redbean code

@gabordemooij gabordemooij merged commit a6af947 into gabordemooij:master Mar 28, 2012

gabordemooij pushed a commit that referenced this pull request Jun 16, 2012

Gabor de Mooij
Merge pull request #141 from damianb/patch-3-cs
CS fixes to match context of other redbean code

@mozey mozey referenced this pull request Jul 25, 2014

Closed

RPDO.php and PHP_INT_MAX #386

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment