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>

Track and report actual displayed size

render:reposition messages now contain more accurate columns
information, and are generated whenever it is updated.

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

display-xcb: switch to embedded data

display-xcb pane now uses embedded data.

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

display-ncurses - switch to embedded data.

Ncurses now uses embedded data in pane

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

core-window: use embedded data

core-window data is now embedded.

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

doc-text - use embedded data in pane

The doc-text pane now embeds the data

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

multipart - use embedded data

doc-multipart now used embedded data.

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

doc-list: use embedded data

doc-list panes now embed the data.

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

doc-dir: switch to embedded data

the pane or doc-dir now embeds the data.

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

log: switch to embedding data in pane

log panes now have data embedded.

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

core-editor: embed ed_info in the struct pane

This makes all content of ed_info just one pointer dereference away from
any pane.  We can start putting content that is current global, into
ed_info.

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

Support inline pane data.

If a NULL data pointer is passed to __pane_register(), then a larger
memory allocation will be made to hold both the pane and the data.

If a type is defined with PANE_DATA_TYPE then that will be the size
passed to __pane_register by pane_register().

Similarly DOC_DATA_TYPE can give a type for a document.  The "struct
doc" must appear first in this type.

If PANE_DATA_TYPE is defined, then "core-pane.h" must be included after
the type has been declared, which will usually be after "core.h" is included.

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

Always handle pane_register failure immediately.

Rather then
 p = pane_register();
 if (p)
     ....

Use

 p = pane_register();
 if (!p)
    return FAIL
 ...

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

Include some include files where there are needed in include files.

This makes it easier to move includes around in .c files.

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

Introduce point_new()

point_new() is vmark_new() for MARK_POINT.

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

Introduce mark_new()

mark_new() is vmark_new() for MARK_UNGROUPED.

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

linecount: allow attachment to a viewer stack.

Sometimes we want linecount to count lines in a view, not in a document.

Instead of having the view pretend to be a document for linecount,
change it to explicitly support attachment.

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

linecount: fix mark used for 'goto'.

The mark passed to CountLines/goto:line as 'mark' is handed to
doc:GotoLine as 'mark2', but that function expects 'mark'.

This works as the doc: function always gets point for mark if nothing
else is given.  But it is wrong. Let's fix it.

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

view: fix scroll bar on directories etc.

When linecount is attached to a view, as with lib-format (directories)
and emails, the location of the scroll bar handle is wrong.

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

Disable pane_too_long() with edlib_timing_allowed = False.

edlib_timing_allowed = False
now disables pane_too_long() too.

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

linecount: run the on-idle event on the linecount pane.

We currently queue the on-idle event on the document.  That is untidy.
The event belongs to the linecount pane, so that is where it should run.

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

emacs: allow :C-q to be repeated when a unicode name was found.

If we found a unicode char with a given name, there might be another
one.
So allow the command to be repeated.

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

Add lib-unicode-names and use it for emacs :C-q

:C-q can now be given a unicode name from NamesList.txt and it will
insert the first match - which might not be the one you want.

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

search: various fixes for searching a string in a string

I guess I hadn't tested this, but searching for a string in a string
didn't work for various reasons.

Also allow max length to search to be given.

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

Use XDG standard to find config (and data) files.

Provide an interface to search for config and data files in appropriate
XDG locations.  Enhance this by looking in the directory that contains
the main shared library - after HOME but before system locations.
Use this for config files.  This removes the need to use ARGV[0]
to find the initial ini file.

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

render-lines: fix vline positioning.

When we want to position the cursor line precisely, we need
consume_space to consume at most one line each top a bottom else we
might miss the target line.

Also document how find_lines() works with a bit more detail.

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

render-lines: don't change y_pre or y_post after found_start/end set

To be able to achieve precise positioning on the display (e.g.  with
vline) we need to stop changing y_pre or y_post once found_start or
found_end are set - respectively.

So when we hit EOF and set a 10% y_post, we should NOT set found_end
yet.

This shows an off-by-one error in vline calculation which was previously
hidden as consume_space() would always move us to the start of that line.

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

history: support searching history of entries

C-S searches forward, but the more useful is C-R which searches
backward.

Search doesn't wrap - I wonder if it should.

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

pygtk: discard new_display2

new_display and new_display2 are now sufficiently similar that we can
merge them.

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

lib-glibevents: catch errors from callbacks.

The on-idle and poll callbacks didn't catch errors, would cause be
fatal.

Also change on-idle callbacks to not return errors, on Efalse.

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

display-xcb: merge display_xcb and xcb_new_display

These two function are largely equivalent, so xcb_new_display does a
couple of step that are not harmful for display_xcb.

So discard display_xcb and just use xcb_new_display.

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

Finish attach-display-foo commands with doc:attach-view

And attach-display-* command must now be called on a document or on the
root.
If it was on a document, that document is viewed in the created display
using doc:attach-view.
This makes it easier for a caller to create a useful window in a single
command.

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

call editor:activate-display from the display code.

Any command that creates a display pane now also calls
editor:activate-display and returns the result.
That means other code what wants a display doesn't need to know about
activating the display.

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

server: don't hold reference to the display pane.

The display pane will soon not be available as creation a display will
transparently stack various things on top of it.
So use "Display:close" instead of .close() and just hold the top level.

.term and .disp are often the same now. I wonder if we need both.

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

add window:set: and use it to set attributes for displays.

We sometimes need to set DISPLAY etc so it is visible to the display
handler, particularly for ncurses.
Use new window:set: to set these on the window-core pane.
This means that editor:active-display is now always immediately after
the display pane is created, so we are closer to letting that be called
as part of creating the display pane.

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

handle Display:close in window-core

When a display/window is closed, the whole stack needs to go including
the window-core pane.  So let the Display:close request get down to
there, and close everything.

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

introduce core-window.c

core-window provides a window-core pane which provides core
functionality for all windows.  Some functionality is moved out of
lib-input where it doesn't really belong.

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

input: don't use pane_notify to send window notification.

input can currently use pane_notify directly instead all calling
window:notify:foo because it manages these notifications.
But soon it won't, so call("window:notify:foo") like everyone else has
too.

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

macro: don't walk up tree to "find" input.

We can send "Keystroke" to the focus and it will find the right target.
We don't need to "find" the target first.  Doing so is fragile.

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

Move parse_ini() into a separate include file.

This makes it easy for it to be reused, or even take out into another
package.  Unlike the rest of edlib, it can be used under any popular
open source license.

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

compose: run notmuch-addr asynchronously.

We can no longer call pane.root.refresh(), so we cannot update the
display while waiting for something - which is good: we shouldn't wait
for things.
So arrange for an even when notmuch-addr reports some data and insert
the first match then - if point hasn't moved.

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

config: allow initial panes to be set in config file.

Changing the initial panes for a display give lots of control to the
user, and particularly will allow alternate key mapping to emacs.

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

Split module list into a separate ini file

One day the list of module dependencies will be mechanically generated
from all available modules.  To prepare for that, and to keep edlib.ini
small, split out modules.ini.

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

discard old auto-load code

Now that lib-config can autoload modules, get rid of the old limited
approach.

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

pane: if an on-idle event to handle damage cannot be added, don't add damage.

If pane_damaged() is called before the event handler is ready, the
on-idle doesn't get registered properly and will never get called.
So check for errors, and leave damage unset so the next pane_damaged()
call can schedule the on-idle event.

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

messageline: right justify the date/time

Do this because we can.  And because I once wanted to.
I wonder if I'll like it.

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

messageline: use lib-renderline for drawing the message line.

This provides more flexibility and already allows long lines to wrap -
as is very obvious in the "spell" test case.

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

messageline: treat an empty message as message-removal.

If a Message command provides a zero-lenth message, remove any current
message rather than setting a explicit empty message.  This allows a
default to appear.

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

renderline: allow Refresh to be handled.

If the requested z depth (at attach time) is >= 0, then Refresh messages
will be sent.
They can now be handled.
The cursor pos for the refresh can now be set with renderline:set.
This ->num is only used for Refresh.

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

renderline: allow z depth to be given when attaching.

Rather than assuming a negative z depth is always wanted, use ci->num
when the pane is attached.

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

lib-markup: generate markup with soh/stx/etc rather then <> etc

There is less likelyhood of problems with stray '<' using this form of
markup.

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

render-complete: handle cursor display properly.

If ->mark2 is passed to doc:render-line, do the right this to report the
offiset.

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

render-complete: add highlight more correctly.

As the original might have attributes in it, we need to find the
character that we want to highlight, and highlight just those.  Possibly
there will be multiple ranges if the characters already have differing
highlights.

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

render-complete: strip new-style attrs when returning result.

Copy strip_attrs() from lib-linefiler and use that to strip all attrs
from line before returning result.

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

lib-linefilter: always strip attributes from line before comparing.

If the key is an attribute, tell the callback using ci->key=="attr"
rather than the magic of -num2 == -1, which can get set by other
doc:render-line functions.

When the key is a line from doc:render-line, properly strip all
attributes, both old and new style.

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

render-present: don't use doc:render-line to get text.

doc:render-line returns a line that can include markup.  But
render-present doesn't expect or handle markup.
So use doc:get-str to get the line, and quote and '<' characters.

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

lib-renderline: make control chars stand out more.

Conrtol chars should not appear in the line given to lib-renderline.
We try to handle them, but let us make them much more obvious so they
look wrong as well as being wrong.

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

lib-renderline: expect control chars instead of <> for attributes.

Using <> to distinguish attributes is clumsy as it requires that '<' be
quoted.

We already quote control characters, making them <red>^X</red> so use
control characters for distinguishing attributes.
SOH (\1) marks start of attrs
STX (\2) marks end of attrs, start of text
ETX (\3) marks end of the text for those attrs.
ACK (\6) is used as a filler when needed.

A string that start \6 or \1 is assumed not to contain meaningful <>
Otherwise we translate to remain compatible with old code.

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