constexpr infinity

constexpr infinity

~/skia (inf) $ nm /tmp/rel/obj/src/core/libskia.SkLiteDL.o | grep GLOBAL__sub
<nothing>

TBR=halcanary@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2228203002

Committed: https://skia.googlesource.com/skia/+/cd8864111d191be4db42a478d6bf877eb50277a7

[mtklein_C, 2016-08-09 23:10:03]

Description was changed from

==========
constexpr infinity

~/skia (inf) $ nm /tmp/rel/obj/src/core/libskia.SkLiteDL.o | grep GLOBAL__sub
<nothing>

TBR=halcanary@google.com
BUG=skia:
==========

to

==========
constexpr infinity

~/skia (inf) $ nm /tmp/rel/obj/src/core/libskia.SkLiteDL.o | grep GLOBAL__sub
<nothing>

TBR=halcanary@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2228203002
==========

[mtklein_C, 2016-08-09 23:10:11]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-08-09 23:10:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mtklein_C, 2016-08-09 23:11:35]

If this doesn't work, I'll just reinterpret_cast a uint32_t constant
(0x7f800000).

[commit-bot: I haz the power, 2016-08-09 23:37:50]

Description was changed from

==========
constexpr infinity

~/skia (inf) $ nm /tmp/rel/obj/src/core/libskia.SkLiteDL.o | grep GLOBAL__sub
<nothing>

TBR=halcanary@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2228203002
==========

to

==========
constexpr infinity

~/skia (inf) $ nm /tmp/rel/obj/src/core/libskia.SkLiteDL.o | grep GLOBAL__sub
<nothing>

TBR=halcanary@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2228203002

Committed:
https://skia.googlesource.com/skia/+/cd8864111d191be4db42a478d6bf877eb50277a7
==========

[commit-bot: I haz the power, 2016-08-09 23:37:51]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/cd8864111d191be4db42a478d6bf877eb50277a7

[hal.canary, 2016-08-10 00:47:10]

lgtm

[reed1, 2016-08-10 11:53:16]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2016-08-10 11:53:17]

https://codereview.chromium.org/2228203002/diff/1/src/core/SkLiteDL.cpp
File src/core/SkLiteDL.cpp (right):

https://codereview.chromium.org/2228203002/diff/1/src/core/SkLiteDL.cpp#newco...
src/core/SkLiteDL.cpp:20: static const SkRect kUnset = {(SkScalar)INFINITY,
0,0,0};
If this is better, should we use this for our definition?