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.

Sign up for GitHub

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

Jump to bottom

Line ending with { is flipped to } #6

Open
wesalvaro opened this issue Jun 25, 2018 · 5 comments · May be fixed by #7
Open

Line ending with { is flipped to } #6

wesalvaro opened this issue Jun 25, 2018 · 5 comments · May be fixed by #7

Comments

@wesalvaro
Copy link

wesalvaro commented Jun 25, 2018 •
edited

Easy 100% repro for me:

$ echo foo > cat
$ echo bar { > dog
$ cwdiff cat dog
foobar }
@wesalvaro
Copy link
Author

Looks like the color_filter is the culprit.

wesalvaro added a commit to wesalvaro/cwdiff that referenced this issue Jun 25, 2018
@wesalvaro
Fixes parsing of wdiff output
d869d91 
Fixes junghans#6
@wesalvaro wesalvaro linked a pull request Jun 25, 2018 that will close this issue
Open
@lemzwerg
Copy link
Contributor

lemzwerg commented May 2, 2024

Ditto for [, which gets flipped to ].

$ echo "[ x ]" > foo
$ echo "x" > bar
$ cwdiff foo bar
]x ]

@junghans
Copy link
Owner

junghans commented May 2, 2024

@lemzwerg I haven't used cwdiff for years, do you want to take it over?

@lemzwerg
Copy link
Contributor

lemzwerg commented May 3, 2024

Alas, I'm swamped with work for LilyPond and FreeType, sorry.

Maybe I'll find some time to check #7, but I can't promise anything.

@junghans
Copy link
Owner

junghans commented May 6, 2024

No worries, I remember that #7 failed on OSX back when, hence I didn't merge it, but I might need to try it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes parsing of wdiff output wesalvaro/cwdiff
3 participants
@wesalvaro @junghans @lemzwerg