Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring of diff algorithm and utilities #20

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

satabin
Copy link
Member

@satabin satabin commented Oct 29, 2017

No description provided.

When implementations are instantiated they do not care what the type of
diffed element will be.
This typeclass intends to avoid conversion to an `IndexedSeq` object,
for classes that are not. It exposes the required API so that the Lcs
algorithms work. In particular, this avoids converting a `String` into
an `IndexedSeq`.
By default two elements of the sequences to diff are considered
equivalent if they are equal in the sense of the `equals` method.
However, by providing a different `Equiv` instance, you may change the
way elements are considered equal.
@satabin satabin added this to the 0.2.0 milestone Oct 29, 2017
@satabin satabin self-assigned this Oct 29, 2017
@codecov-io
Copy link

codecov-io commented Oct 29, 2017

Codecov Report

Merging #20 into master will increase coverage by 1.68%.
The diff coverage is 71.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   38.63%   40.31%   +1.68%     
==========================================
  Files          43       46       +3     
  Lines        1522     1580      +58     
  Branches      167      171       +4     
==========================================
+ Hits          588      637      +49     
- Misses        934      943       +9
Impacted Files Coverage Δ
.../main/scala/gnieh/mustache/MustacheProcessor.scala 0% <ø> (ø) ⬆️
src/main/scala/gnieh/pp/SimpleDoc.scala 80% <ø> (ø) ⬆️
src/main/scala/gnieh/regex/tdfa/TNfa.scala 0% <ø> (ø) ⬆️
src/main/scala/gnieh/diff/Diff.scala 0% <ø> (ø) ⬆️
src/main/scala/gnieh/regex/tdfa/TDfa.scala 0% <ø> (ø) ⬆️
src/main/scala/gnieh/pp/Renderer.scala 60% <ø> (ø) ⬆️
src/main/scala/gnieh/matching/Bitap.scala 0% <0%> (ø) ⬆️
src/main/scala/gnieh/diff/Lcs.scala 96.49% <100%> (+3.5%) ⬆️
src/main/scala/gnieh/diff/DynamicProgLcs.scala 100% <100%> (ø) ⬆️
src/main/scala/gnieh/diff/MyersLcs.scala 95% <100%> (ø) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4638f6...b8e1f94. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants