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

Already on GitHub? Sign in to your account

Fixes the --column and context case. #242

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

shlomif commented Dec 12, 2012

Hi @petdance and all.

After some investigation I was able to fix the problem. A large part of the problem was that the newly added tests were broken and I didn't notice it.

See:

petdance#240

Please pull. All existing tests pass.

Regards,

-- Shlomi Fish

pvinis and others added some commits Oct 25, 2012

Fix tests & code for --column + -C.
The tests and the code for the --column and -C context case from the
previous commits were broken. This commit fixes both the code (to not
emit the warnings) and the tests (so the right behaviour will be
exhibited).

See:

#240 .
Collaborator

hoelzro commented Aug 28, 2013

Moved the issue to ack2; the patch will have to be as well.

@hoelzro hoelzro closed this Aug 28, 2013

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