[SERVER-21290] remove pessimizing moves Created: 04/Nov/15 Updated: 31/Mar/16 Resolved: 17/Nov/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | None |
| Fix Version/s: | 3.2.0-rc4 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Eric Milkie | Assignee: | Kaloian Manassiev |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Sprint: | Sharding C (11/20/15) | ||||||||
| Participants: | |||||||||
| Comments |
| Comment by Githook User [ 17/Nov/15 ] | ||||||||||||
|
Author: {u'username': u'kaloianm', u'name': u'Kaloian Manassiev', u'email': u'kaloian.manassiev@mongodb.com'}Message: | ||||||||||||
| Comment by Eric Milkie [ 11/Nov/15 ] | ||||||||||||
| ||||||||||||
| Comment by Eric Milkie [ 04/Nov/15 ] | ||||||||||||
|
Clang 3.7 is now the default version on Fedora 23, so I expect we'll be adding such a compiler to our build armada soon. | ||||||||||||
| Comment by Mathias Stearn [ 04/Nov/15 ] | ||||||||||||
|
Clang 3.7 catches it. At the moment that is (was) the only one. We did a pass and scrubbed the others a while ago. The only reason this one remained is that it is in a unittest so isn't picked up when compiling core. | ||||||||||||
| Comment by Spencer Brody (Inactive) [ 04/Nov/15 ] | ||||||||||||
|
Is this something coverity could be catching? | ||||||||||||
| Comment by Githook User [ 04/Nov/15 ] | ||||||||||||
|
Author: {u'username': u'RedBeard0531', u'name': u'Mathias Stearn', u'email': u'mathias@10gen.com'}Message: |