ExportXMLWordPrintableJSON

    • 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

            Assignee:
            Unassigned
            Reporter:
            Johnathan Martell
            None
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: