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>

menu: allow memu to get wider

If menu gets smaller we already reduce size.
Now we do so when menu gets bigger too.
For this, we add 1 to width so there is a change for it to appear wider
as the width reported is only ever the visible width.

This makes all menus slightly wider.

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

notmuch: focus on query pane when changing messages

The previous patch didn't fix this properly.  'K-a' wasn't handled for
examine.
Try again.

Fixes: 5707b369a802 ("notmuch: when moving among messages, set focus to query pane.")
Signed-off-by: NeilBrown <neil@brown.name>

history: improve stability of search.

When searching for string that matched at start of line, we would stay
on the line as we typed more of the match.
When searching elsewhere, we keep moving backward in the history.

Fix by always starting search at end of line.

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

popup:  fix popup_child_notify

Two silly typos meant this didn't work properly.

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

render-lines: fix cursor-placement sometimes off-screen

I was measuring against the bottom of the cursor, not the top.

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

wiggle: skeletal support for IgnoreBlanks

I don't know if I would ever want IgnoreBlanks, but this shows some of
the changes that would be needed for it to work.

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

mergeview: highlight result after change.

When an alt-m command replaces the merge with some result, highlight the
result to help user see exactly what it is.

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

wiggle: nul-terminate string from collect_merge()

I love untested code.

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

lib-abbrev: Don't show a menu when not needed.

If there is only one match, just insert it, don't show a menu.

If all matches have a common prefix, just insert that.

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

make: possibly fix problem with not always following.

This is just a guess, but maybe this is why make sometimes doesn't
follow the output properly.

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

.gitingore updates.

Added these targets and forgot to tell git.

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

doc-test: doc:clear must send doc:replaced

Without this, linecount gets badly confused.

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

render-lines: improvements to left shift.

If there is lots of space on the right, only shift left if the cursor
requires it.

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

render-lines: remove obvious error

There can be no justification to multiply by cw twice.

This doesn't affect display-ncurses but is problematic for x11

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

render-lines: ensure cursor doesn't fall off end of line.

If the line is not allowed to left-shift and doesn't wrap, cursor might
be off the right edge.
In that case, do think that the cursor is drawn.

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

TODO - lots of updates

So much to do, so little time.

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

lib-utf8: use do_char_byte()

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

qprint: use do_char_byte()

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

lib-crop: use do_char_byte()

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

crop: switch to embedded pane data

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

base64: use do_char_byte

I duplicated the code.. I wonder if I should have done.

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

mulitpath: use do_char_byte()

Just a simple conversion using mp_step largely unchanged.

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

render-format: use do_char_byte()

This requires some more ugly #define stuff, but I think the code sharing
is good.

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

render-format: switch to embedded pane data

pane data is now embedded.

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

doc-email use do_char_byte for the doc:char function

do_char_byte now passes a mark to the next/prev functions and doc-email
makes use of it.

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

doc-email: switch to embedded data.

struct email_view is now embedded in the pane, no found by a pointer deref.

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

emacs: don't move cursor when opening/selecting menu.

Having the cursor jump around looks noisy, so try removing it.

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

Move-CursorXY - only perform action on direct hit.

If Move-CursoyXY don't find a character under the cursor, don't perform
the action.  Still move the mark though.

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

Always pass 0 to Move-CursorXY for num and num2

The num and num2 values aren't used, and the values passed aren't
consistent.
The new meaning of "str" takes over what some of them were meant to do.

So always pass 0.  Maybe one day I'll think of something useful to do.

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

Change how Move-CursorXY is passed "no change" co-ordinates.

Currently a negative co-ordinate means "use the ->cx/cy value of the target
pane".  This allows changing position in one dimension only.

However it prevents receiving a negative co-ordinate, which might happen
in the mount event happens out-side the pane which has grabbed the
mouse.

So instead of a negative number, use MAX_INT to say "don't change".

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

lib-menu: prohibit left-shift of display.

As the menu window is sized to match the longest line, there is a risk
of a left-shift happening if the cursor ever ends up at the end of that
line.
That would be bad, so explicitly disable left-shift.

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

lib-abbrev: fix positioning of menu

The xy location of the menu must be relative to the parent of the menu.

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

do_char_byte: pass the pane to DOC_NEXT, not doc_data

If we pass the pane, the inline code is more generic and does not need
to know about the internals of struct pane.
This is a constant offset from the doc_data, so does not change the
number of memory accesses.

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

doc-docs: switch to doc_data and do_char_byte()

Use embedded doc in pane, and use common code for doc:char handler.

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

Simplify some doc:char handlers.

Create a shared inline function that does the bulk of the work of a
'doc:char' function and only needs to be given next and prev functions
which update the doc_ref and report the character.

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

doc-text fix text_next for bytes

I was completely ignoring 'bytes' for text_next.
Consequently the 'hex' test, which display bytes, was all wrong.

Also simplify text_prev using denormalise

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

Fix return of Action

If Action doesn't find an action tag it must return Efallthrough so some
other pane has a change to handle K:Enter

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

notmuch: remove support for saved: synonym for query:

Before notmuch supported query: I was handling saved:.
For a while I've supported both, but that adds no value.
So get rid of all saved: search terms and saved. config entries.

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

notmuch: support mouse-click to select search and thread

mouse-click on search name will select that search.
mouse-click on thread subject will select that thread,
or message within a thread (I think... that last one is a bit weird).

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

Update copyright dates.

Gotta keep those dates accurate :-)

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

Add lib-askpass

AskPass will as for a password and not display what is entered.
A new script "el-askpass" is provided that ssh can use to ask for a
password through edlib.
lib-shellcmd sets the environment so this script is used.

Net result is that if a command run from edlib needs ssh to ask for a
password, it can.

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

lib-server: separate out choose_pane functionality

Create function to choose the display and pane on which to place the popup

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

Add a "bin" type for xdg-find-edlib-file

This searches $PATH after looking the a "bin" dir near the lib that the
code was loaded from.

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

Make a "bin" directory

"bin" will hold program that can be run.

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

server: split client and server code more cleanly.

Have all server code together, and all client code too.

Also differentiate based on argv[0] - it seems a little cleaner.

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

lib-server: various clean-ups.

- Use '\0' to separate arg and env, rather than space.
- Use arg consistently - not ':' etc.
- Use simpler names for commands, don't try to mimic internal messages.

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

autospell: provide drop-down menu on spelling errors.

Multiple enhancement to make menus work as drop-downs triggered by a
mouse action.  Still some problems.

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

Revise how location-sensitive actions are managed.

To create a location-sensitive action, the attribute "action-FOO" must
be set to a command name, for some value of FOO.
Currently only "activate" is effective, and this send on a mouse click.
In the future "menu" may be added, and possibly others such as for
drag-and-drop.

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

render-lines: update lines/cols earlier

Instead of only updaes lines/cols during the final render, update them
when choosing the range of lines to display.
This makes them available to interested parties earlier.

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

Initial menu support.

We can now place a menu and receive a selection.
This is demonstrated with lib-abbrev which pops up a menu showing the
options to choose between.

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