-
Type:
Improvement
-
Resolution: Unresolved
-
Priority:
Minor - P4
-
None
-
Affects Version/s: None
-
Component/s: js-bson
-
Not Needed
-
None
-
None
-
None
-
None
-
None
-
None
-
None
Use Case
As a driver engineer
I want to use more concise code when searching for null terminators
So that code is more maintainable
User Experience
- External refactor ticket for findNull results in less code to maintain, which is also simpler to read
Dependencies
- None
Risks/Unknowns
- Small behavioural change in this PR in that negative offsets no-longer walk up to the null terminator. Likely desirable behaviour.
Acceptance Criteria
Implementation Requirements
- findNull loop is refactored to be more readable and maintainable by removing unnecessary bound check and collapsing increment.
Testing Requirements
- None
Documentation Requirements
- None
Follow Up Requirements
- None