| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
More info can be found here: https://github.com/halostatue/diff-lcs/issues/77
|
| | |
|
| | |
|
| |
|
|
|
|
| |
We have ai, bj to keep track of which index we're in for both a
and b, so keeping i is not needed
Without it, the logic will be simpler
|
| | |
|
| |\
| |
| | |
Fix off-by-one error
|
| | |
| |
| |
| |
| | |
in ruby, (0..matches.size) will include both sides in the range, which
means we'll get off-by-one error
|
| |/ |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
# This is the 1st commit message:
Fix improperly placed chunks
Resolve #65
- Also add even more tests for checking `ldiff` results against `diff` results.
- Fix issues with diff/ldiff output highlighted by the above tests.
- Add a parameter to indicate that the hunk being processed is the _last_ hunk;
this results in correct counting of the hunk size.
- The misplaced chunks were happening because of an improper `.abs` on
`#diff_size`, when the `.abs` needed to be on the finding of the maximum diff
size.
# This is the commit message #2:
Ooops. Debugger
# This is the commit message #3:
Restore missing test
- Fix some more format issues raised by the missing test.
- Start fixing Rubocop formatting.
# This is the commit message #4:
Last RuboCop fixes
# This is the commit message #5:
Finalize diff-lcs 1.4
# This is the commit message #6:
Fix #44
The problem here was the precedence of `or` vs `||`. Switching to `||` resulted
in the expected behaviour.
# This is the commit message #7:
Resolve #43
# This is the commit message #8:
Typo
# This is the commit message #9:
Resolve #35 with a comment
|
| |
|
|
|
| |
- Required to fully support rspec.
- Resolves #63.
|
| |
|
|
|
|
|
| |
- This appears to satisfy the issues found in #60 as well as providing the
additional fixes required to properly test ldiff.
Resolves #60
|
| |
|
|
|
|
|
| |
Resolves #57.
- The `diff_size` should be an absolute value.
- Added a test to ensure that the issue does not introduce a regression.
|
| |
|
|
|
| |
- Support hoe >= 3.18
- Various bug fixes and updates
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
- This required some level of code remediation in the main library, but all
tests now pass:
- Patchsets are now internally flattened one level explicitly, rather than
using Array#flatten. This ensures that only the outer patchset array is
flattened.
Fixes #48.
|
| |
|
|
|
| |
- Other linting configuration also applied.
- Soft-deprecating versions older than 2.3.
|
| |
|
|
|
|
|
|
|
|
| |
This reverts commits 01e0cae and 3b4d2be because they introduce fatally
breaking changes in the tests that were hidden by broken CI.
* 01e0cae: Add #to_ary to Diff::LCS::Change and Diff::LCS::ContextChange
[Akinori MUSHA]
* 3b4d2be: Mention in rdoc that Diff::LCS::ContextChange can be converted to an
array [Akinori MUSHA]
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
- Updated testing and gem infrastructure.
- Cleaning up documentation.
- Modernizing specs.
- Silence Ruby 2.4 Fixnum deprecation warnings. Fixes #36, #38.
- Ensure test dependencies are loaded. Fixes #33, #34 so that specs can be run
independently.
- Fix issue #1 with incorrect intuition of patch direction. Tentative fix, but
the failure cases pass now.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Diff::LCS::Hunk could not properly generate a difference for
comparison sets that are not US-ASCII-compatible because of the use of
literal regular expressions and strings. Jon Rowe (JonRowe) found this
in rspec/rspec-expectations#219 and provided a first pass
implementation in diff-lcs#15. I've reworked it because of test
failures in Rubinius when running in Ruby 1.9 mode. This coerces the
added values to the encoding of the old dataset (as determined by the
first piece of the old dataset).
https://github.com/rspec/rspec-expectations/issues/219
https://github.com/halostatue/diff-lcs/pull/15
* Adding Travis CI testing for Ruby 2.0.
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
- Only on 1.8.7. Found by travis-ci.org.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
- Bringing indents in a little.
- Simplifying a little logic so that boolean shortcuts are taken faster.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. All Diff::LCS::__* methods are now moved to Diff::LCS::Internals and
named without the underscores.
2. Fixed the documentation for Diff::LCS::patch. (Some formatting may
have been broken.)
3. Eliminated normalize_patchset in favour of the (substantially
similar) analyze_patchset. This allows for a single-pass over a
patchset to determine whether the patchset contains changes and
transform it to the object-based format.
4. Made Diff::LCS::ContextChange descend from Diff::LCS::Change. This
will probably change so that both descend from something like a
Diff::LCS::AbstractChange because of some issues that may arise with
case statement order.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Fixed some formatting and style issues.
- Trailing spaces
- Calling class methods using '.' instead of '::'.
- Resolved Issue #2 by handling string[string.size, 1] properly (it
returns "" not nil).
- Added special case handling for Diff::LCS.patch so that it handles
patches that are empty or contain no changes.
- Adding temporary code to help determined the reason for the
misidentification of patch direction.
- Added a number of different specs to check for comparing the same
value.
- Added broken spec filtering.
|
| | |
|
| |
|
