rfc : -Wshorten-64-to-32 warnings in folly liger dependencies
authorRanjeeth Dasineni <ranjeeth@fb.com>
Wed, 20 Aug 2014 19:38:57 +0000 (12:38 -0700)
committerAnton Likhtarov <alikhtarov@fb.com>
Fri, 26 Sep 2014 22:20:40 +0000 (15:20 -0700)
commit6fc7a48c7e3a5cdb0c97196b2ff0f08c6ee073d7
treeabbfeca5d997e0f7271db30675c9c0832fd6dc36
parent54ab6858b79c1bea35e5dcba593b60d4f92b58f3
rfc : -Wshorten-64-to-32 warnings in folly liger dependencies

Summary:
This is not a fix but me seeking advice here. The context here is that in liger (our common network
stack for mobile platforms), we saw bugs due to truncation and want to enable these warnings. To find those in our code,
I had to first resolve the folly ones. I just got around by making the truncation explicit so that I can get to real errors.
Ideally I would like owners to examine each case and fix the original/accept the explicit truncate if its intended. Let me
know what you think. The last (least preferred) is for us to keep this as a liger only change. We have a couple of ways to
do that but it would be nice to fix.

N.B : this covers only liger dependencies

Test Plan: errors resolved after these changes.

Reviewed By: njormrod@fb.com

Subscribers: trunkagent, doug, shilin, njormrod, seanc, pgriess

FB internal diff: D1509355
folly/Conv.h
folly/Format-inl.h
folly/Hash.h
folly/String-inl.h