DeepDiffs.jl icon indicating copy to clipboard operation
DeepDiffs.jl copied to clipboard

Deduplicate longest common subsequence logic

Open goretkin opened this issue 4 years ago • 1 comments

This is pretty much purely an aesthetic suggestion to deduplicate the logic (prompted because I had to change == -> isequal in two places in https://github.com/ssfrr/DeepDiffs.jl/pull/14 . This PR depends on that PR.)

I feel like the changes lie on a spectrum. For that reason, I broke the change out into a million commits. If the code at the last commit doesn't look so great, perhaps one just before is better. Also, each commit mostly represents a straightforward transformation.

goretkin avatar May 10 '20 04:05 goretkin