emacs-search: be careful to avoid infinite loop.

Assuming that 'm' moved if length > 1 isn't entirely safe.
It *should* be the case, but protecting against infinite loops
deserves belt *and* braces.

So add some extra checks, and test for a case that currently causes an
infinite loop

Signed-off-by: NeilBrown <neil@brown.name>

textfill: get prefix from regexp when appropriate.

Revise code for finding start of para so that if the regexp identifies a
prefix - rather than a whole line - we use that prefix to decide where
para starts, rather than skipping punctuation.

Also include trailing spaces of a prefix in the prefix for .md files.

Signed-off-by: NeilBrown <neil@brown.name>

edlib.ini: discard stray \ quotes

The content found in .ini files does not support \ quoting, so the \\
results in to '\' which we don't want.

Signed-off-by: NeilBrown <neil@brown.name>

rexel: support -B for backtrack testing.

Add -B option and improve usage() message.

Signed-off-by: NeilBrown <neil@brown.name>

Review Welcome text

*Welcome* text was getting old and irrelevant.

Also clean out all remaining triage, as they are all done.

Signed-off-by: NeilBrown <neil@brown.name>

diff: use correct file name in 'pre' mode.

When in 'pre' mode we need to go to the 'pre' file, not the post one.
So get the filename after '---'.

Also re-arrange some from the triage in the TODO file.

Signed-off-by: NeilBrown <neil@brown.name>

Change window-scaling to display-scaling

The emacs scaling commands now affect the whole display (window), not
just a single pane.  This is more consistent.

It is done just by changing an attribute on the display pane.

Signed-off-by: NeilBrown <neil@brown.name>

Introduce generic Display:set:

Display:set: can be used to set any attribute on the display pane.
The attr name can be in str2, or following Display:set: in the key.

Signed-off-by: NeilBrown <neil@brown.name>

Change Display:set-noclose to Display:set:no-close and store in attribute.

Using an attribute to store the no-close string simplifies code, and
using "Display:set:" is similar to "doc:set:" and will soon be used to
set any attribute at the display level.

Signed-off-by: NeilBrown <neil@brown.name>

Get rid of attributes shift_left and shift-left

Add more meaning to render:wrap and discard shift-left and shift_left.
It was confusing having two ...  or three ...  attributes for almost the
same thing.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: fix problems with finding position of 'click' location.

This broken menus and links etc.  So add a test for menus in the
spelling test.

Signed-off-by: NeilBrown <neil@brown.name>

Allow command name to be used to choose some data files.

The icon file, Welcome text, and initial config file are now chosen
based on the base name of the command being run.
So if "edlib" is copied to "elma", then "elma.ini" is used for config,
and "elma-icon.png" for an icon.
If the expected file isn't found, a file named for "edlib" is tried
instead.

This should make it easy to have have various commands providing
different interfaces.
 elma - EMACS mode
 elvi - VI mode
 elnm - notmuch email reader
 elpnt - presentation mode
 eled - editor with A binding.
maybe.

Signed-off-by: NeilBrown <neil@brown.name>

Move welcome text into a separate file.

This makes it easier to edit the welcome text, and to have different
welcome text for different editors.

Signed-off-by: NeilBrown <neil@brown.name>

display-x11: load icon if available.

If edlib-icon.png can be found, load it as the icon for the X11 window.

Signed-off-by: NeilBrown <neil@brown.name>

Move config file into new 'data' directory.

Also make lib/edlib point to data, not just to '..'.
This will allow the image files to be found with xdg-find-edlib-file/data.

Signed-off-by: NeilBrown <neil@brown.name>

Add a logo

Currently it is just a few image files. Will do more one day.

Signed-off-by: NeilBrown <neil@brown.name>

Introduce "Cancel" commands - K:ESC in emacs

"Cancel" is like "Abort", but softer.
Currently it cancels any active selection.

Signed-off-by: NeilBrown <neil@brown.name>

mode-emacs: improve path completion in shell commands

1/ Allow a ':' or '=' to identify where the path starts as long as it
   starts '/'.
2/ Give message when directory cannot be opened or no completions can be
   found.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: use foreach_attr() to parse image attrs.

This isn't a big win, but it is a bit nicer.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: discard cached-size attribute

We don't need an attribute for this, just store it in rline_data.

Signed-off-by: NeilBrown <neil@brown.name>

Fix "edlib -g"

Obviously this was never tested last time I made a change.

Signed-off-by: NeilBrown <neil@brown.name>

tests: add markup test

This test uses the new test-markup pane.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: create a new render-item whenever tab: is set.

I sometimes want tab: with no text - particularly to set wrap-margin.
So allow that.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: enhance cvt() to allow > in text attributes.

This is not particularly useful, but I wanted a '>' in a wrap-head for
testing, and I couldn't.  So I made it possible.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: change right: margin to measure +ve from left.

now right-margin works like left and tabs.  +ve is from left edge, -ve
is for right edge.
0 is different. It is right-edge for right margin, and left-edge for others.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: allow word-wrap setting via attributes.

Word-wrap is normally enabled for the whole pane.  But making it
possible for individual lines might be useful, and certainly eases
testing.

Signed-off-by: NeilBrown <neil@brown.name>

tests: Allow extra data files to be used in tests.

Only files in tests.d that start with a digit are now considered to be
tests.
Other files are copied into DOC/extra in the test area for tests to
examine.
I put it DOC so that most tests don't see the extra entry in a directory
and so don't need to be changed.  Only 02-presenter looks in DOC

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline; fix double-cursors in display-x11

When a line wraps, display-x11 is showing cursor at end of one line and
start of next.  Fix that.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: fix up scaling of margin, vspace, tabs

left and right margins, and above/below space, and tab stops must be
scaled based on the font size - which we measure early.  That already
includes the ->scale factor, so we mustn't include it again.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: handle garbage better in cvt()

cvt() shouldn't assume a well-formatted string, and must always quite
when it sees EOL.

Signed-off-by: NeilBrown <neil@brown.name>

display-ncurses: ignore focus changes better.

If we got a focus change message, 'buf' would be uninitialised and used.
So initialise it - and don't use it.

Signed-off-by: NeilBrown <neil@brown.name>

display-ncurses: Improve attr parsing

Move the attr parsing code from lib-renderline into core-misc.c and
use it in display-ncurses to avoid using a fixed-size char array that we
never test for overflow.

Signed-off-by: NeilBrown <neil@brown.name>

add lib-test-markup

lib-test-markup allows arbitrary markup to be provided to lib-renderline
for testing.
When enabled with "test-markup", each line of the document is treated as
verbatim markup

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: always give cursor offset when measuring cursor line

As presence of cursor can change size of appearance, we need to always
report where the cursor is when measuring the line.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: another wrapping fix.

If the wrap item is the part that won't it, need need to be caseful that
'x' hasn't been adjusted for xdiff yet.

Signed-off-by: NeilBrown <neil@brown.name>

tests: fix up "extend" and extend 00-basic

We now have a test in 00-basic which moves the cursor in and out of a
wrap field so that it appears and disappears.  When it appears it
changes the number of displayed lines.  This doesn't work properly.

Signed-off-by: NeilBrown <neil@brown.name>

history: add support for "favourites".

favourites can be configured for different history files.
A-n moves to favourites if not in history.

config file can now have "doc:FOO" sections for docs that don't have
file names.

A bug in doc:EOL is fixed

Signed-off-by: NeilBrown <neil@brown.name>

lib-render: move tab/non-tab distinction out of find_xy()

The special handling of tabs is best kept in common code.
So move more into do_measure().

Signed-off-by: NeilBrown <neil@brown.name>

render-line:measure now reports if the line ended properly

If the last line in a document doesn't end with a newline character, we
need to handle that interesting case.
Make it easier to do so.

Signed-off-by: NeilBrown <neil@brown.name>

merge-view: allow 9:A-m to both cut and paste.

If there is an  active merge 9:A-m cuts it.
If there isn't but is a selection, 9:A-m will paste it.

Now if there is an active merge and a selection, 9:A-m will cut and
paste.

Signed-off-by: NeilBrown <neil@brown.name>

mergeview: adding merge to region needs full-lines in region

If the region isn't whole lines, make it so.

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: avoid putting extra space at EOF for small pane

We normally add a bit of extra space at end-of-file.
I already had code to avoid this in small panes. But it wasn't working
as line_height_post hadn't been initialised usefully.

Signed-off-by: NeilBrown <neil@brown.name>

emacs-search: don't try to replace same string twice

:A:Enter replaces but stays in same place.  Repeating the command causes
the replacement to be attempted again, which is bad if orig and new are
different length.
So detect and prevent this.

Signed-off-by: NeilBrown <neil@brown.name>

README: delete all "Done" items.

It was all getting too noisy.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline - lots of cleanup

Review all non-image code and clean things up in various ways.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: only measure line when needed

Before performing measure_line work, check if any of the inputs have
changed.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderlines: clean up left_margin handling.

In particular the wrap_margin must initially be based on the left margin.

Signed-off-by: NeilBrown <neil@brown.name>

rexel: fix bug with capturing groups.

The post-parse editing of group matches to put all the right branches
in place was messing up capture requests.

Signed-off-by: NeilBrown <neil@brown.name>

messageline: full right-align date/time

Now that I can fully right-align without wrapping, do it for the time in
the message line.

Signed-off-by: NeilBrown <neil@brown.name>

tests: add a faster version of refresh.

Doing it "fast" often works, so allow that as an option.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline allow rtab to avoid line wraps.

If rtab is used, don't wrap if last char is at right edge.
If no rtab, continue to wrap so there is somewhere for cursor to be.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: various wrapping fixed.

In particular, the subject line in notmuch summary lines wraps properly
now.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: Fix length of split section in do_draw()

Need to subtract start from the end to get the length.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderlines: erase before drawing an image

I lost the Draw:clear for images, so they get drawn on the wrong pane
and end up as background.

Signed-off-by: NeilBrown <neil@brown.name>

TODO updates.

What still needs to be done for lib-renderline...

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: move tab handling into do_measure()

This makes do_measure() more like do_draw()

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: pass ri to do_measure()

Pass in the ri, and don't bother passing the attr which is always
ri->attr.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: introduce measure_str() for simple string measurement.

This makes a clean separation leaving do_measure() only for text in
->line.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: don't pass scale to do_measure()

instead of passing scale, extra rd from p and use that to find scale.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderlines: pass split to do_draw() instead of str/len/tab_cols.

I think this removes most of the ugliness.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline; pass ri to do_draw instead of ri->attr

I'll use more from ri soon, for now just pass it in and use the ->attr.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: factor out draw_wrap()

Drawing the line-wrap markers doesn't need all the complexity of
do_draw(), so factor it out into a separate function.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: add ->ascent inside do_draw()

Instead of all callers of do_draw() needing to add ->ascent,
add it inside do_draw().

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: drop scale arg to do_draw()

We can find the scale from rline_data which we can find from the pane.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline change rl vars to ri

rl vars which used to be for render-line are now for render-item,
so rename to ri.

Signed-off-by: NeilBrown <neil@brown.name>

massive lib-renderline rewrite

This now passes my tests - after adjusting some tests.

lib-renderline is totally different.  Parsing and processing are now
distinct steps.  Hopefully this will make it easier to work with in
future, though it is still very rough in places, particularly with
handling for \t tabs.

I need to go through it several times and clean stuff up, but at least I
now have a credible base to work on.

Signed-off-by: NeilBrown <neil@brown.name>

renderline - fix margin for tab.

For some strange reason, tabs were wrapping before other chars.

Signed-off-by: NeilBrown <neil@brown.name>

Don't show default wrap tail for requested wrap point.

Requested wrap points suggest wrap is expected, so don't show the
wrap tail.  I think it is ugly.

Signed-off-by: NeilBrown <neil@brown.name>

tile: silence use-before-set warning.

"remain" is only used if "remaining" is not zero, so it will be set
before used.  But compiler cannot see that.  Help it out.

Signed-off-by: NeilBrown <neil@brown.name>

messageline: don't ask for cursor on render-line

To avoid asking of a cursor, we need to pass -1.

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: pass focus to vmark_set()

Good practice is to always send the focus.
We will need this to get the prefix at 'set' time.

Signed-off-by: NeilBrown <neil@brown.name>

history: use A-r rather than C-R for search

Using C-R and particularly C-S break search-replace because I can no
longer search-again from the replace box (which has history).

So switch to A-r A-s.

Signed-off-by: NeilBrown <neil@brown.name>

lib-input - don't assume Efallthrough is 0

One day I might change Efallthrough..

Signed-off-by: NeilBrown <neil@brown.name>

ncurses: catch and ignore focus-in/out

focus-in/out comes in as unknown key at awkward times causing problems
for C-l and the refresh causes a focus-out.
So catch and ignore for now.
Also log any unrecognised keys so I can try to handle them.

Signed-off-by: NeilBrown <neil@brown.name>

Change pane_resize() to return Bool

pane_resize() now reports if any change happened.
One caller wants this.

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: factor out code in revalidate_start()

revalidate start considers 3 cases when the point is on the line.  Each
of these checks if the point is on the line.
Factor out that test and other code so that 3 (or 4) cases can focus on
their particular issue.

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: call_render_line_to_point never returns error

Don't test for call_render_line_to_point returning < 0, as it doesn't.
Don't even test for == 0.  That is a valid answer and we no longer call
call_render_line_to_point() in a situation where that causes problems.

Signed-off-by: NeilBrown <neil@brown.name>

render-lines: don't look for point in line where it isn't.

call_render_line_to_point() doesn't return an error, so calling
it when point isn't on the line isn't well defined.
So don't do it.

This changes the number of blank lines after end-of-file, which I think
I am happy with, so self-tests change.

Signed-off-by: NeilBrown <neil@brown.name>

lib-server: fix a couple of typos

I mustn't be using this code enough because these should be obvious.

Signed-off-by: NeilBrown <neil@brown.name>

minor unalloc improvements.

unalloc can now free 'const' pointers, and strings.

Signed-off-by: NeilBrown <neil@brown.name>

renderline: change render-line:measure to return the cursor attributes.

Rather than saving the attributes at the cursor to be returned later,
return them directly from render-line:measure.

Signed-off-by: NeilBrown <neil@brown.name>

renderline: change render-line:findxy to return the attributes

Rather then remembers the attributes to be collected later with a
get:attr, return the attributes in a comm2 callback for immediate use.

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: change to use embedded pane data

The rline_data is now embedded in the pane.

Signed-off-by: NeilBrown <neil@brown.name>

minor change to xcb drawing and scaling

1/ ask for "monospace", not "mono"
2/ default to 12 point
3/ scale up and down by same ratio.  use 1.2 like others
4/ fix font_description size calc, which was weird.

Signed-off-by: NeilBrown <neil@brown.name>

compose-email: check for needed headers:

Don't sent the messages unless there is at least
 - From:
 - Subject:
 - To: or Cc:

Signed-off-by: NeilBrown <neil@brown.name>

discard commcache.

The idea here was to accelerate repeated calls to the one request, so we
didn't need to repeat the search up the pane tree, or for the comm in
the keymap.
However I since decided that it is generally best to avoid repeated
calls like this, and mostly we do.

Signed-off-by: NeilBrown <neil@brown.name>

Avoid null pointer deref when ccache used.

This can make ->home NULL which times_up() no trips over.

I should probably discard this ccache stuff.

Signed-off-by: NeilBrown <neil@brown.name>

emacs: support shows shell output in a popup.

With a prefix arg of 0 or 1, shellcmd still interpolates the output into
the current buffer.
With a prefix arg of 4, it now shows the output in a popup window, like
visiting a file with
  Cx 4 4 f

Signed-off-by: NeilBrown <neil@brown.name>

Factor out list sorting into list.h

No point having two copies of the same code.

Signed-off-by: NeilBrown <neil@brown.name>

renderline: calculate curs_width even when only a prefix is displayed.

Also make sure that the size we shift is always at least 1 - else it
loops forever.

Signed-off-by: NeilBrown <neil@brown.name>

mode-emacs: allow cx-} and cx-{ to be repeated with single char.

Just like cx-< etc

Signed-off-by: NeilBrown <neil@brown.name>

lib-renderline: fix handling of truncation.

When looking for a bug in wrap-mode I found this bug in non-wrap mode.
But I cannot reproduce the first bug any more.
Weird.

Signed-off-by: NeilBrown <neil@brown.name>

Don't initialise ->data in pane_init()

There may not be a ->data any more, and the caller initialises this if needed.

Signed-off-by: NeilBrown <neil@brown.name>

Rename _CALL _VFUNC _VFUNC_

All names that start underscore followed by capital letter are reserved
by C library.  So stop using them.

Signed-off-by: NeilBrown <neil@brown.name>

Avoid using names with "__" prefix.

Names prefixed with "__" are reserved for use by the C library.
For global names, use "do_" instead.
For local name, use a single "_".

Signed-off-by: NeilBrown <neil@brown.name>

Add doc:list-sort and sort command name for completion.

doc:list-sort can sort entries in a doc-list based on an attribute of
choice.

emacs command listing uses this.

Signed-off-by: NeilBrown <neil@brown.name>

emacs: use doc-list for command name completion.

doc-list is good for this sort of thing.

Signed-off-by: NeilBrown <neil@brown.name>

Change times_up() to use pane_too_long() on root.

Also check for debugger in all pane_too_long() calls.

Signed-off-by: NeilBrown <neil@brown.name>

Don't impose time limits when run under a debugger

Single-stepping in the debug can make things take much longer.  We
shouldn't let that change behaviour or debugging becomes harder.
So detect debugger and skip the timing.

Signed-off-by: NeilBrown <neil@brown.name>

compose-email: use menu for address completion

Rather than having 'tab' cycle through addresses, drop down a menu to
choose from.

Signed-off-by: NeilBrown <neil@brown.name>