Browser: improve rendering of conflicts in the the 'merge' view.

I was ignoring the in_conflict value, so the 'original' side
of a conflict wasn't properly displayed, and so looked confusing.

Signed-off-by: NeilBrown <neilb@suse.de>

Browse: don't use background colors for added/deleted.

Coloured backgrounds are too loud.
So for added and deleted use red and green text.

Need to find a way to highlight lone spaces..

Signed-off-by: NeilBrown <neilb@suse.de>

Browse: avoid display problems when changing view-mode.

If we change mode so that the current end-of-line is no longer
visible, the display gets confused and add extra lines.  So when mode
changes, move backward then forward to ensure we are on a visibile
line.

Signed-off-by: NeilBrown <neilb@suse.de>

isolate_conflicts: don't look for eol that cannot be there.

When looking for 'wiggles' we need to track whole lines
in the original file.  But when looking for an end-of-line
we shouldn't look at an merge-item that doesn't even
exist in the original file.

Reported-by: Chris Peterson <cpeterso@cpeterso.com>
Signed-off-by: NeilBrown <neilb@suse.de>

Don't include endian.h

It turns out it isn't needed, and OS/X doesn't have it.

Reported-by: Chris Peterson <cpeterso@cpeterso.com>
Signed-off-by: NeilBrown <neilb@suse.de>

browser: display the type of the merge-node under the cursor.

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: allow cursor to be displayed in alternate pane.

This allows us to navigate all of the merge.
When in the alternate pane, movement is based on visibility there.

Signed-off-by: NeilBrown <neilb@suse.de>

browse/search: support selectively ignoring case.

We ignore case only if no upper chars.
Leave possiblity for explicit request for case-match later.

Signed-off-by: NeilBrown <neilb@suse.de>

browse/search: don't match strings in the hunk-header.

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: Support searching-again within the same line.

This requires mcontains to know which direction we are searching in,
and whether we want a new match, or whether the old one will do.

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: change searchdir to -1 or +1

instead of 1 or 0.  It is less confusing that way.

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: Extend proper cursor tracking to search.

Record the cursor in search anchors so we can go back to the
right location.

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: Use KEY_BACKSPACE to recognise a backspace.

It isn't always control-H

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: add a more effective cursor.

Create a proper cursor structure to track where on the screen we are.
Record the width of the current symbol so 'backward' works better.
Allow movement master start and end of line by going to previous or
next line.
Allow 'search' to locate the actual string, not just the line.

Signed-off-by: NeilBrown <neilb@suse.de>

browse: fix buffer underflow when closing directories.

Signed-off-by: NeilBrown <neilb@suse.de>

Add another regression-test

This is the test case that lead to the recent context improvements
and others.

Signed-off-by: NeilBrown <neilb@suse.de>

Initial all of 'End' merger.

This avoids invalid mem access when checking for ends_line at End.

Signed-off-by: NeilBrown <neilb@suse.de>

Avoid buffer overflow when checking for hunk header

Signed-off-by: NeilBrown <neilb@suse.de>

merge: avoid including large Unmatched sections in a conflict.

A large Unmatched section can just distract from the real conflict
and doesn't give useful information.
So if we can close the conflict before a big Unmatched,
do so.

Signed-off-by: NeilBrown <neilb@suse.de>

Encourage more context into conflict reports.

Conflict reports currently don't contain much surrounding
'Extraneous' text.  When the patch didn't really apply at all,
this context is very useful.
So encourage at least 3 lines of context from Extraneous or
Unchanged texts.
If it mostly comes from Unchanged text it isn't quite so useful,
so reduce the requirement.

Update self-tests to match.  In most cases these are an improvement.
In a couple of case we maybe get a bit more context than really
necessary, but that is better than not enough.

Signed-off-by: NeilBrown <neilb@suse.de>

merge: factor out is_cutpoint.

We perform the same test several times, so make a function for it.

Signed-off-by: NeilBrown <neilb@suse.de>

Improve handling of hunk-header when merging.

If we are merging a patch, the patch will contain a number of
hunk-headers (essentially the @@ NN,MM @@) lines.

These should never appear in the merge, and must always line up with
end-of-line in the original as they represent a number of lines having
been deleted.

So:
  Cause these to each create a simple Extraneous merger, never
    combined with adjacent texts

  exclude them from any conflicts in isolate_conflict, so they
    can never appear in a merge

  Ensure that a preceeding 'Unmatched' finishes on an end-of-line.

Also update lots of self tests with improved results.

Signed-off-by: NeilBrown <neilb@suse.de>

Ensure that a hunk-header never appears in the middle of an Extraneous range.

A hunk header is always Extraneous.  However Extraneous text before it is
quite separate and is best kept separate.
Start a new Extraneous merger with each hunk header will make it easier
to detect hunk headers and handle them better.

Signed-off-by: NeilBrown <neilb@suse.de>

browser: clear the end of the split-window when needed.

We would get some garbage when near the end of the diff.

Signed-off-by: NeilBrown <neilb@suse.de>

Add --show-wiggle flag.

--show-wiggle modifies --merge so that chunks which need to be
wiggled into place are presented as full conflicts with an
extra stanza which shows what the resulting wiggle would look like.

Suggested-by: Alessandro Salvatori <sandr8@gmail.com>
Signed-off-by: NeilBrown <neilb@suse.de>

Handle some errors better in Browse mode.

If the target file is missing or the patch is corrupt,
we really don't want to just crash.

Signed-off-by: NeilBrown <neilb@suse.de>

Make tracing code a bit cleaner and more organised.

 WIGGLE_TRACE=1 ./wiggle ....

will turn on tracing.

Signed-off-by: NeilBrown <neilb@suse.de>

Introduce xmalloc and use it.

Instead of sometimes calling die() when malloc fails and
sometimes not, introduce xmalloc() which always reports a message
and dies, and use it uniformly.

Signed-off-by: NeilBrown <neilb@suse.de>

Refactor do_diff slightly

Combine come code that is common to 'word' and 'line' branches.

Signed-off-by: NeilBrown <neilb@suse.de>

Better error message if split_patch fails for --extract

Signed-off-by: NeilBrown <neilb@suse.de>

Split do_diff_words and do_diff_lines out from do_diff

Signed-off-by: NeilBrown <neilb@suse.de>

Minor cleaning up in man page wiggle.1

Signed-off-by: NeilBrown <neilb@suse.de>

Remove extraction of base-name of command.

We don't use this any more (as use of the name 'vpatch' was discarded)
so don't bother to extract it.

Signed-off-by: NeilBrown <neilb@suse.de>

Split do_merge out from main() as a separate function.

Signed-off-by: NeilBrown <neilb@suse.de>

Split do_diff() out from main()

Signed-off-by: NeilBrown <neilb@suse.de>

split 'extact' out of 'main' into a separate function.

Signed-off-by: NeilBrown <neilb@suse.de>

Replace hash.h with better hash function from CCAN

Signed-off-by: NeilBrown <neilb@suse.de>

Fix 'make version'

Since I change the format for the version string, this hasn't worked.

NeilBrown <neilb@suse.de>

.gitignore updates

Signed-off-by: NeilBrown <neilb@suse.de>

General cleanups

- spelling fixes
- remove ifdefed-out code
- white space adjustments
- etc

Signed-off-by: NeilBrown <neilb@suse.de>

ChangeLog-typo

option --quite should read --quiet

Test condition was being ignored

Signed-off-by: Neil Brown <neilb@suse.de>

Silences warnings about set-but-not-used variables.

Signed-off-by: NeilBrown <neilb@suse.de>

Removed malloc.h, fixed off-by-one in bestmatch.c, it was silent in Linux and deadly on Mac OS X.

Signed-off-by: Neil Brown <neilb@suse.de>

Stuff from months ago that I forget why

But it looks ok

Signed-off-by: NeilBrown <neilb@suse.de>

Remove merge.c

It hasn't been used for a while

Signed-off-by: NeilBrown <neilb@suse.de>

vpatch cleanup

add some comments and general clean ups.

Signed-off-by: NeilBrown <neilb@suse.de>

Fix various typos in help messages.

I've obviously not proof-read this...

Signed-off-by: NeilBrown <neilb@suse.de>

Clean up bestmatch.c

Add lots of comments, remove some cruft etc.

Signed-off-by: NeilBrown <neilb@suse.de>

Clean up split.c

Add/fix comments, remove unused function parameter.

Signed-off-by: NeilBrown <neilb@suse.de>

Comment and tidy up diff.c

Add lots of comments explaining algorithm. Change some
variable names and minor improvements.

Signed-off-by: NeilBrown <neilb@suse.de>

Clean up more valgrind errors.

Various issues found by watching valgrind.  Typically falling
one way or the other off the end of an array.

Signed-off-by: NeilBrown <neilb@suse.de>

Use Cmd var instread of "wiggle" in various error messages.

Signed-off-by: NeilBrown <neilb@suse.de>

Fix white space etc issues reported by checkpatch.pl

i.e. checkpatch.pl from the linux kernel.

Signed-off-by: NeilBrown <neilb@suse.de>

Update date COPYING and FSF address.

Just addresses and license commentary change.  No change in license.

Signed-off-by: NeilBrown <neilb@suse.de>

Fix use of uninitialised vars in merge2.

This makes valgrind happier.

Signed-off-by: NeilBrown <neilb@suse.de>

Avoid use of undefined variable in bestmatch.

When v->val is <= 0, v->k doesn't mean anything and isn't needed, so
avoid using it.  This helps make valgrind happy.

Signed-off-by: NeilBrown <neilb@suse.de>

nul terminate stream read from a file.

Though we carry around the length of a string and so will not normally
need a nul terminator, we sometimes use sscanf which cannot take a
length and so needs nul termination.

So add a nul when loading from a file or stdin.

Signed-off-by: NeilBrown <neilb@suse.de>

dotest: Check if /usr/bin/time supports the needed functionality.

Not all version of /usr/bin/time support -o and -f.  So test if the
installed one does and act accordingly.

Reported-by: Oliver Kiddle <oliver@kiddle.eu>
Signed-off-by: NeilBrown <neilb@suse.de>

Stop using case ranges.

case a...b:
is gcc specific.  So don't use it - do an explicit range test instead.

Reported-by: Oliver Kiddle <oliver@kiddle.eu>
Signed-off-by: NeilBrown <neilb@suse.de>

Ask for more warnings and fix some.

In particular we were using strncmp without including string.h -
because memory.h automatically includes it..

Reported-by: Oliver Kiddle <oliver@kiddle.eu>
Signed-off-by: NeilBrown <neilb@suse.de>

Make lots of functions "static".

In particular isolate_conflicts() needs to be static to avoid a link
error with SUN's C compiler
Reported-by: Oliver Kiddle <oliver@kiddle.eu>
Also this shows that set_prefix() wasn't being used any more, so fix
that.

Signed-off-by: NeilBrown <neilb@suse.de>

Makefile portability patch

Don't run tests with sh, since sh is not always bash.

Signed-off-by: Neil Brown <neilb@suse.de>

Change format of 'version' string

Based on suggestions from  Jari Aalto <jari.aalto@cante.net>

Signed-off-by: NeilBrown <neilb@suse.de>

More options for giving patches to --browse

As well as a 'patch' file (which identifies the target file by
names in the patch file), you can now also browse given a
target file plus a diff,
just the diff if it ends '.rej',
a target file plus old and new versions of the file.

Signed-off-by: NeilBrown <neilb@suse.de>

ReadMe.c - spelling fix

Signed-off-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Neil Brown <neilb@suse.de>

Use git-archive to create a release

it avoid unwanted files.

Signed-off-by: NeilBrown <neilb@suse.de>

Write some notes about editting merges.

browse: allow selective listing of only files with wiggles or with conflicts.

Signed-off-by: Neil Brown <neilb@suse.de>

browser: add help window

Add display of help window on '?', and add
lots of help text for various modes.

Signed-off-by: Neil Brown <neilb@suse.de>

browser: rearrange code

Put all the 'main window' code together, and all the
'merge window' code together.

Signed-off-by: Neil Brown <neilb@suse.de>

browser: first cut at horizontal split.

When in 'merge' mode we display the patch in a bottom window when on a
conflict or wiggle, so full context can be see.

This is a bit rough at the moment - the image jumps more than I would
like and it is hard to visually follow what is happening.

Signed-off-by: NeilBrown <neilb@suse.de>

browser: prev/next_melmnt to take mp not mpos.

As prev/next_melmnt only use the main 'struct mp' of an
mpos, it is less confusing to only give them that to work on.

Signed-off-by: Neil Brown <neilb@suse.de>

browse: never leave blank space above

Having blank space above the display looks
ugly, so if some appears, refresh to remove it.
Also get 'control-L' to recenter as well as refresh.

Signed-off-by: NeilBrown <neilb@suse.de>

browser: Minor search improvements

We still don't get control-s through though.

Signed-off-by: Neil Brown <neilb@suse.de>

browser: move check_line call

instead of calling before eac call to draw_mline, call it
inside draw_mline instead.

Signed-off-by: NeilBrown <neilb@suse.de>

Add some tracing to browser

Allow examining each element/pos in the current context

Signed-off-by: NeilBrown <neilb@suse.de>

Browser: add line number tracking.

Useful for matching against linenos given in patch.

Signed-off-by: NeilBrown <neilb@suse.de>

browser: display patch chunk headers in 'diff' output.

When showing 'before' of 'after' text which comes from a patch,
display the chunk header:
   @@ xx,yy +aa,bb @@

Signed-off-by: NeilBrown <neilb@suse.de>

Significant browser update.

wiggle -B
is getting closer to what I wanted.  Lots of dead code removed
and lots of fine details tidied up.
Usage is a bit different in that we don't always see a split
window.

Various sundry changes like .gitignore and "make demo.path"

Signed-off-by: NeilBrown <neilb@suse.de>

FORTIFY_SOURCE fixes

Fix a buffer overflow, some unused values, and some uninitialised variables.

Signed-off-by: NeilBrown <neilb@suse.de>

Release 0.8

Added some patches to resolve issues raise for openSUSE package.

Add --no-ignore option

This suppresses the optimisation of ignoring conflicts which appear to
be making a change that had already been made.

Signed-off-by: NeilBrown <neilb@suse.de>

Improve parsing of diff3 output.

When diff3 discovers the the 1st and 3rd files match, but differ from
the 2nd it reports:

<<<<<<<
content from second file
=======
content from 1st and 3rd file
>>>>>>>

without a ||||||| line at all.

Enhance the extract code to recognise this.

Signed-off-by: NeilBrown <neilb@suse.de>

Update wiggle.spec for new version and location.

Release version 0.7

After nearly 7 years...

vpatch: silence a compiler warning.

Apparently attrset seems to return a value so we need
to explicitly ignore it.

Signed-off-by: NeilBrown <neilb@suse.de>

Add a number of tests which had not been commited to 'git' previously.

Signed-off-by: NeilBrown <neilb@suse.de>

Improve diff/fixup when diff is greedy at end of a diff.

The 'fixup' code to make up for the greediness of diff did not
treat the end of the diff as a (zero length) match which can be grown.
This could leave no context at the end of a 'diff'.

Applying a patch with no trailing context can cause confusion.

This fixes the issue raise by Debian Bug 271766

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=271766

The sample from that bug is included as a test.

Signed-off-by: NeilBrown <neilb@suse.de>

Makefile: (clean): do not delete original files

Signed-off-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Neil Brown <neilb@suse.de>

extract.c: (split_merge): initialize with NULL

Signed-off-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Neil Brown <neilb@suse.de>

wiggle.1: fix hyphens

Signed-off-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Neil Brown <neilb@suse.de>

Improve parsing of diff -u output.

Normally chunks start
  @@ -%d,%d +%d%d @@
but if a section is just one line long, the ',1' is dropped, so
  @@ -%d +%d @@

is possible.

Fix an off-by-one bug.

I assume .... I don't remember, sorry.

vpatch: remove a pointless dereference.

Various little changes to p.

I did them a while ago an don't remember..

Fix a couple of little bugs that crept in to p

Signed-off-by: Neil Brown <neilb@suse.de>

Add export option to p

This is a preliminary attempt at git integration.

Signed-off-by: Neil Brown <neilb@suse.de>

Add rebase function to 'p'

This moves all 'applied' patches to 'included'
and makes the current files 'current'  and 'orig'.

Signed-off-by: Neil Brown <neilb@suse.de>

When calling a recursive p, use $0 to find it.

This applied to recommit and decommit

Signed-off-by: Neil Brown <neilb@suse.de>