Add typing for diff opcodes.

Review Request #14418 — Created May 2, 2025 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

This change updates diff opcodes to use explicitly typed tuples. The
opcodes created by the Differ implementation have a tag and ranges,
and then the opcode generator adds an additional metadata field. In the
interest of avoiding any changes, we therefore have separate definitions
for both steps.

  • Did a bunch of smoke testing of diff generation and saw that
    everything still worked correctly.
  • Ran unit tests.

Diff Revision 4 (Latest)

orig
1
2
3
4

Commits

First Last Summary ID Author
Add typing for diff opcodes.
This change updates diff opcodes to use explicitly typed tuples. The opcodes created by the `Differ` implementation have a tag and ranges, and then the opcode generator adds an additional metadata field. In the interest of avoiding any changes, we therefore have separate definitions for both steps. Testing Done: - Did a bunch of smoke testing of diff generation and saw that everything still worked correctly. - Ran unit tests.
f8ad232f1ebd19142135457de29d0b3156da9519 David Trowbridge
reviewboard/diffviewer/chunk_generator.py
reviewboard/diffviewer/differ.py
reviewboard/diffviewer/myersdiff.py
reviewboard/diffviewer/opcode_generator.py
reviewboard/diffviewer/processors.py
reviewboard/diffviewer/smdiff.py
reviewboard/diffviewer/tests/test_myersdiff.py
Loading...