• Jim Meyering's avatar
    folly: ProducerConsumerQueue: avoid many -Wsign-compare errors · 7fdf8141
    Jim Meyering authored
    Summary:
    There were many templates instantiated with signed types.
    Changing them to unsigned avoids the compilation failures
    noted below.  However, we'll have to watch arc-diff-spawned
    tests for new failures, just in case.
    
    This is by far the largest undecomposable patch of those
    I've written for this task so far.  It is possible that
    some subset of these changes accomplishes the same goal,
    but once I managed to get a successful compilation, I have
    only re-reviewed it once and did not try to shrink it.
    
    Test Plan:
    Without these changes, we would get the following 32 errors from gcc-4.9:
    https://phabricator.fb.com/P19692642
    With them, (and all of the other patches attached to the parent task,
    all of fbcode compiles and passes its tests (modulo preexisting
    failures and aborts).
    
    Reviewed By: delong.j@fb.com
    
    Subscribers: trunkagent, folly-diffs@
    
    FB internal diff: D1773461
    
    Tasks: 5941250
    
    Signature: t1:1773461:1420821964:61635f3a2efc95e8d4457c36e7a5ee3a0a50df23
    7fdf8141
ProducerConsumerQueue.h 4.89 KB