ipc: Convert k(u)int*max types from basictypes.h to std::numeric_limits variants.

ipc: Convert k(u)int*max types from basictypes.h to std::numeric_limits variants.

As the types from basictypes.h are deprecated this moves them to
std::numeric_limits from <limits> or when necessary to the (U)INTx_{MIN,MAX} macros from <stdint.h>.

BUG=138542
TEST=ipc_tests
R=tsepez@chromium.org

Committed: https://crrev.com/7023f52b1ae1a47e3764cf5e7ad8ce65c73c0e54
Cr-Commit-Position: refs/heads/master@{#348474}

[tfarina, 2015-09-09 18:03:27]

Next I will convert basictypes.h includes to macros.h.

PTAL.

[tfarina, 2015-09-10 18:11:28]

Ping?

Copying Carlos, just in case.

[Tom Sepez, 2015-09-10 18:36:24]

On 2015/09/10 18:11:28, tfarina wrote:
> Ping?
> 
> Copying Carlos, just in case.
I'm not sure you are going to be able to make this work fully, owing to the
non-const nature of max() on win.  I was waiting til you got a clean compile
before proceeding.  Thanks.

[cpu_(ooo_6.6-7.5), 2015-09-10 21:30:06]

cpu@chromium.org changed reviewers:
+ cpu@chromium.org

[cpu_(ooo_6.6-7.5), 2015-09-10 21:30:06]

what Tom said

[tfarina, 2015-09-11 04:34:15]

On Thursday, September 10, 2015, <tsepez@chromium.org> wrote:

> On 2015/09/10 18:11:28, tfarina wrote:
>
>> Ping?
>>
>
> Copying Carlos, just in case.
>>
> I'm not sure you are going to be able to make this work fully, owing to the
> non-const nature of max() on win.  I was waiting til you got a clean
> compile
> before proceeding.  Thanks.

Please, don't wait for compile failures to let me know. I have no way to
know if you have looked or not at the patch if you don't send me a message.
So if you are letting me handle the win failures, just leave a short
message saying that you reviewed but saw failures. I haven't had time yet
to look at the win trybot failures. I will see tomorrow, but I was
counting/hoping on your review to help with any issues.

Best,


>
> https://codereview.chromium.org/1331673002/
>


-- 
Thiago Farina

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Tom Sepez, 2015-09-11 15:15:53]

> Please, don't wait for compile failures to let me know. I have no way to
> know if you have looked or not at the patch if you don't send me a message.
> So if you are letting me handle the win failures, just leave a short
> message saying that you reviewed but saw failures. I haven't had time yet
> to look at the win trybot failures. I will see tomorrow, but I was
> counting/hoping on your review to help with any issues.
> 
> Best,

No worries. I had just figured that you'd look at the compile logs, and in 15
seconds realize that what you were trying to do was intractible, and eitehr
close it out or revise drastically, so there wasn't any point in reviewing given
the impending revisions required.

[tfarina, 2015-09-11 17:16:44]

PTAL. I changed it to use the macro types from stdint.h, hopefully that will fix
the win failures.

[Tom Sepez, 2015-09-11 17:40:17]

Great. LGTM.

[tfarina, 2015-09-11 18:30:40]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-09-11 18:31:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1331673002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1331673002/20001

[commit-bot: I haz the power, 2015-09-11 19:58:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-11 19:59:48]

Patchset 2 (id:??) landed as
https://crrev.com/7023f52b1ae1a47e3764cf5e7ad8ce65c73c0e54
Cr-Commit-Position: refs/heads/master@{#348474}

[commit-bot: I haz the power, 2015-09-23 12:25:51]

Patchset 2 (id:??) landed as
https://crrev.com/7023f52b1ae1a47e3764cf5e7ad8ce65c73c0e54
Cr-Commit-Position: refs/heads/master@{#348474}