[SERVER-7143] Replace other integer parsing code with calls to parseNumberFromString() Created: 24/Sep/12 Updated: 27/Jun/19 Resolved: 27/Jun/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | 2.2.0 |
| Fix Version/s: | 4.3.1 |
| Type: | Task | Priority: | Minor - P4 |
| Reporter: | Andy Schwerin | Assignee: | Nathan Brown (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | neweng, platforms-re-triaged | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Dev Tools 2019-06-17, Dev Tools 2019-07-01 |
| Participants: |
| Description |
|
We have uses of atoi, atol, strtol, and I believe at least one custom a-to-i implementation in our codebase. Switching to parseNumberFromString(), which has actual unit tests and easy-to-follow error logic would be a good idea. |
| Comments |
| Comment by Githook User [ 27/Jun/19 ] |
|
Author: {'name': 'Nathan Brown', 'email': 'nathan.brown@10gen.com'}Message: |
| Comment by Githook User [ 27/Jun/19 ] |
|
Author: {'name': 'Nathan Brown', 'email': 'nathan.brown@10gen.com'}Message: |