tlist_for_each{,_entry}_continue* no longer need head of list.

We can use the type information to stop at the right place.

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

Remove unused function point_reset

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

simplify/improve mark-at-point functions.

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

Pass pane to doc_set_attr instead of using pt->doc;

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

Remove ->owner field from point.

No longer needed.

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

Points: allocate list of links separately.

It was just too much hassle having a single allocation and managing
a single pointer to a point.

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

Remove ci->pointp

Now completely gone.

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

Remove pointless doc and pointp from emacs_file_complete and emac_doc_complete

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

emacs_findfile: use pane_attr_get instead of doc_attr.

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

emacs_kill_doc - no pointp

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

Remove last pointp from render-lines

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

Stop using pointp for Move-View-Pos

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

complete_return doesn't need pointp either.

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

Change complete_set_prefix to no need direct access to point/doc

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

Remove pointp usage from emacs insert/delete commands.

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

Remove pointp usage from a lot of Move-* calls.

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

Remove doc arg from mark_to_mark.

It isn't really neded.
We can easily remove doc from doc_next_mark_all() as it isn't used.
We can also have a version of doc_prev_mark_all() that only gets
called when we *know* there is a previos mark, as we do ni mark_to_mark.

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

Send point as ci->mark for events from keyboard or mouse.

I am deprecating ->pointp, but sometimes I need that mark.

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

Don't set pointp for "Replace"

no replace function uses it anymore.

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

Change "Refresh" function to pass 'point' via ci->mark.

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

Change render-line command to accept point in 'mark2' instead of 'ptp'.

And get rid of point pointp stuff.

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

Remove pointp arg for render-line-prev

We only needed the doc, and we can get that from the pane.

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

Stop using pointp in text_reundo and text_replace.

use home->data and home->point instead.

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

text_reundo: rename 'm' to 'm2'.

I want to introduce a more global 'm'.

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

Move final doc default actions from 'view' to 'doc'.

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

Improve default move-file

Instead of moving one char at a time, just jump to the right end.

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

Now that move_X is in core-doc, remove the pointp references.

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

move simple word/char/eol/file movement from 'view' to 'doc'.

I think it makes more sense here.

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

remove two pointp users from emacs-search.

Just use doc_replace to trigger a refresh.

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

Remove render-dir and render-text

Increasingly dead-weight.  Now that we have render-lines and
render-format, these can go.

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

use doc_get_pane to get document for render-format movement.

Removes another pointp user

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

Remove pointp from render_hex_eol.

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

Use panes and marks to communicate with CountLines

so no more pointp;

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

Do for dir_close what we did for docs_bury.

It really is the same thing.

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

Remove pointp setting from mark_same_pane

completely pointless

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

Remove pointp usage from text_save_file and text_get_str

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

Notfiy:Replace doesn't need pointp any more.

So discard it.

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

Use "Notify:Replace" for notifications.

We currently use "Notify" both to make changes and to notify
of changes.
This can be confusion - they are different tasks needing different names.

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

docs_destroy - avoid pointp.

Use doc_get_pane instead.

Maybe I should be sending a message....

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

Change doc:destroy to work with pane rather than point.

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

Remove pane_point.

Only one called remains, so open-code it there.

That caller should eventually go too.

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

Remove dependence of search on pane_point.

This requires PointDup being able to provide a simple mark,
and a new function to find the document pane.

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

"Close" no longer needs pane_point.

render_lines was the last users, and it now uses panes for vmarks.

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

Change vmark functions to take a pane instead of a document.

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

Change vmark_matching to take a pane rather than a doc.

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

Render-attach functions no-longer need the pane_point.

So discard it.

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

Change doc_{add,del,find}_view into commands.

they get send to the document instead of having to find it.

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

Register keymaps early rather than late.

The more we use them, the more careful we must be.

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

Change mark_same to key off tile rather than point.

Thus remove pointp reference from render-lines-notify

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

Have all "render-line" function fail cleanly if point/mark are NULL.

This makes it a bit cleaner to find the render-line pane with a void call.

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

Provide a "Move-to" command to use e.g. when completing a search.

Better than direct access to point_to_mark().

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

Change default_render from a struct field to an attribute.

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

Use Move-File to move to start of a newly created document.

This is cleaner than moving the point directly

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

text: fit bug with text deletion.

I wasn't quite leaving things is a good state.
I think this is better - as this ref no longer over-takes
a follow ref that won't be relocated.
But I really need to review this code.

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

emacs: use pane_final_child rather than pane_point.

This is (now) a suitable place to attach attributes.

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

Introduce doc:set-name

Use this variously in mode-emacs to avoid pane_point calls.

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

Remove pointp from render_format_attach

Not used.

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

Use the new "ThisPane" to find pane to load new doc/file.

Makes the code nicer too.

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

Introduce ThisPane

This is similar to "OtherPane" but find the current tile.

popup uses this if it want to make a popup that is over a doc instead of
over the display.

This removes another use of ->point

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

Introduce PointDup command

This make it possible to get a dup of point without
explicitly finding the point. Makes it more indirect.

You this in 'search' to avoid hunting up the tree.

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

Don't make new point directly in doc_attach

sometimes we want point_new sometimes point_dup

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

Split a 'doc' pane aways from the 'view' panes.

They really are different things.

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

use new doc_attach_view for attaching "view"

This is a common task and it is good to centralize it.

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

Split doc_attach_view from doc_open.

It is silly combining these two functions with very different results.

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

use pane and mark to pass document to doc:set-attr

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

Change doc:get-attr to use a pane rather than a point.

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

Use doc_notify_change rather than point_notify_change were possible.

This gives more flexibility going forward.

Make sure doc_notify_change is just as fast when a point is provided.

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

Use pane rather then point to pass doc to doc:step

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

Use pane instead of point to pass doc to "Release"

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

Change doc:set-ref to take a pane rather than a point.

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

Change doc_find to return a pane.

That fits the pattern better.

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

Remove the document list and use a list of panes instead.

The 'null display' (ed->root.focus) is now the parent of multiple
panes, each of which owns a document (->data) with a backlink
(doc->home->data == doc).

Now in many places we pass around this pane instead of a point when we
just want to identify a document and not a location in the document.
This means we don't need to allocate points so much, or fake up
temporary ones.

doc_attach() must be used at document creation to attach a document to
the tree.

So
 - "*Documents*" watches this list of siblings rather than the original list.
 - commands can be sent to these panes using key_handle_focus.  Some
   doc commands can be sent to any pane.
 - doc creation returns the pane of the document in ci->focus
   rather than a fake point
     e.g. same_file, load_file

 - pane_attach() takes a pane, not a point.
 - doc_open no-longer needs a gross hack
 - editor_choose_doc returns a pane, and so doesn't need a point.
 - popup holds the doc pane, not a point.

etc.

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

Change 'editor' to be the root of the pane tree.

Displays are all children of that root.

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

Change doc:reundo and doc:get-str to use the pane-path.

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

send "doc:replace" up the pane path

Reduces need for passing points and documents around.
There will be more of this.

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

text: fix bug with refresh when text added to start of first line.

The return value didn't match the comment...

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

text: fix undo/redo bug.

we move the point ref forward, and them move along the mark list
to match.
But we need to go *beyond* any mark at the same position
else when we step the ref forward we won't know to keep going....
It is confusing, but this doesn't crash.

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

Remove the doc_operations structure complete.

After all, it is empty now.

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

Make 'step' a command.

This is the most-used command, I really have to optimise this one day.

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

Make mark_same into a command.

Having two marks and a point in cmd_info isn't good.
Maybe I can get rid of one later.

This is probably quite inefficient.  Using the '2' version
might help.

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

Introduce mark_order_not_same

it seems to have lots of potential users.

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

Use mark_ordered_or_same once more.

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

->replace is now doc:replace

I wonder if it should boe "Replace" ??

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

make get_attr a command.

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

Change set_attr into a command.

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

Change ->reundo from doc_operation to command

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

Initial save-file support.

C-x C-s now saves the file if there is a file name.
No errors are reported if anything goes wrong.

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

Add DEF_LOOKUP_CMD

This makes it easy to handle the common case of a dispatcher command.

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

rename mainloop.c to edlib.c

There is nothing loopy about this file..

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

Change set_ref to be a command instead of directory_op

More messyness with passing documents - time to fix that.

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

Move some common mark list initialisation to a single function.

This results in only one place that calls ->set_ref

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

Change ->destroy from doc_operation to command.

It is a bit messy passing a bare doc to a command at the moment,
but I have plans...

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

Allow doc_destroy to destroy any left-over points

 ... like it does with marks.

Seemed like a good idea...

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

change get_str from a doc_operations to a command.

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

Change doc_load_file and doc_same_file from doc_operations to commands.

Eventually I hope for all doc_operations to disappear.

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

Remove point_ref

it is unused and not needed.

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

Remove mark_of_point

it is a hang over from when I though 'point' might be a private
data structure.

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

Add lib/ to .gitignore

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

render-lines: draw cursor on inactive panes.

This feels a big hacking, but it is nice to know where the cursor is.

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

render-lines: make sure cursor appears in empty file.

when point is at the end of a line it is also at the start of the next
line, and this can cause confusion.

Resolve by preferring the start of the next line (even if it is the
non-existent line at end of file) unless the end of the previous line
in not in the first column (e.g. no newline at end-of-file).

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