[SERVER-20664] WiredTiger and zlib trigger clang 3.7's Wshift-negative-value warning Created: 28/Sep/15 Updated: 07/Dec/16 Resolved: 09/Oct/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Build |
| Affects Version/s: | None |
| Fix Version/s: | 3.2.0-rc0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Mathias Stearn | Assignee: | Mathias Stearn |
| Resolution: | Done | Votes: | 1 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Sprint: | QuInt A (10/12/15) | ||||||||
| Participants: | |||||||||
| Description |
|
It is included in Wall so we'll need to either fix them or suppress the warning when building those libraries.
|
| Comments |
| Comment by Githook User [ 12/Oct/15 ] |
|
Author: {u'username': u'RedBeard0531', u'name': u'Mathias Stearn', u'email': u'mathias@10gen.com'}Message: This is just cherry picking the following two commits from the zlib repo: https://github.com/madler/zlib/commit/e54e1299404101a5a9d0cf5e45512b543967f958 Avoid shifts of negative values inflateMark(). The C standard says that bit shifts of negative integers is ------- https://github.com/madler/zlib/commit/44ae761dc2047038dc42a7e5a162d042fc392d2b Clean up portability for shifts and integer sizes. |
| Comment by David Hows [ 29/Sep/15 ] |
|
This has been fixed in zlib's development branch |
| Comment by Mathias Stearn [ 28/Sep/15 ] |
|
zlib commits: https://github.com/madler/zlib/commit/e54e1299404101a5a9d0cf5e45512b543967f958 |