From 6108d210d508e754904c46f687adbc87ab98dc78 Mon Sep 17 00:00:00 2001 From: Vincent Sanders Date: Sat, 23 May 2020 22:59:40 +0100 Subject: remove all direct access to selection structure --- desktop/selection.c | 62 +++++++++++++++++++++++++++++++---------------------- desktop/selection.h | 35 +++++++++++++----------------- 2 files changed, 51 insertions(+), 46 deletions(-) (limited to 'desktop') diff --git a/desktop/selection.c b/desktop/selection.c index 1fac7e225..06cfdae8f 100644 --- a/desktop/selection.c +++ b/desktop/selection.c @@ -22,32 +22,17 @@ * implementation of text selection within browser windows. */ -#include -#include #include -#include -#include -#include -#include "utils/log.h" -#include "utils/utf8.h" -#include "utils/utils.h" -#include "netsurf/form.h" -#include "html/box.h" -#include "html/box_inspect.h" -#include "html/private.h" -#include "html/font.h" -#include "text/textplain.h" +#include "netsurf/clipboard.h" #include "netsurf/browser_window.h" +#include "netsurf/window.h" +#include "utils/utils.h" +#include "content/content_protected.h" -#include "netsurf/mouse.h" #include "desktop/browser_private.h" -#include "netsurf/plotters.h" -#include "desktop/save_text.h" -#include "desktop/selection.h" -#include "netsurf/clipboard.h" -#include "netsurf/window.h" #include "desktop/gui_internal.h" +#include "desktop/selection.h" struct selection_string { @@ -334,7 +319,7 @@ selection_click(struct selection *s, top = browser_window_get_root(top); - if (selection_defined(s)) { + if (s->defined) { if (idx > s->start_idx) { if (idx <= s->end_idx) { pos = 0; @@ -376,7 +361,7 @@ selection_click(struct selection *s, } else if (mouse & BROWSER_MOUSE_DRAG_2) { /* adjust selection, but only if there is one */ - if (!selection_defined(s)) { + if (!s->defined) { return false; /* ignore Adjust drags */ } @@ -396,7 +381,7 @@ selection_click(struct selection *s, } else if (mouse & BROWSER_MOUSE_CLICK_2) { /* ignore Adjust clicks when there's no selection */ - if (!selection_defined(s)) { + if (!s->defined) { return false; } @@ -521,7 +506,7 @@ bool selection_copy_to_clipboard(struct selection *s) /* exported interface documented in desktop/selection.h */ -void selection_clear(struct selection *s, bool redraw) +bool selection_clear(struct selection *s, bool redraw) { int old_start, old_end; bool was_defined; @@ -539,6 +524,8 @@ void selection_clear(struct selection *s, bool redraw) if (redraw && was_defined) { selection_redraw(s, old_start, old_end); } + + return was_defined; } @@ -569,9 +556,11 @@ selection_highlighted(const struct selection *s, unsigned *start_idx, unsigned *end_idx) { - /* caller should have checked first for efficiency */ assert(s); - assert(s->defined); + + if (!s->defined) { + return false; + } if ((end <= s->start_idx) || (start >= s->end_idx)) { @@ -583,3 +572,24 @@ selection_highlighted(const struct selection *s, return true; } + +/* exported interface documented in desktop/selection.h */ +bool selection_active(struct selection *s) +{ + return s->defined; +} + +bool selection_dragging(struct selection *s) +{ + return s->drag_state != DRAG_NONE; +} + +bool selection_dragging_start(struct selection *s) +{ + return s->drag_state == DRAG_START; +} + +void selection_drag_end(struct selection *s) +{ + s->drag_state = DRAG_NONE; +} diff --git a/desktop/selection.h b/desktop/selection.h index 4f3e8b002..86ff395a1 100644 --- a/desktop/selection.h +++ b/desktop/selection.h @@ -20,8 +20,8 @@ * Text selection within browser windows (interface). */ -#ifndef _NETSURF_DESKTOP_SELECTION_H_ -#define _NETSURF_DESKTOP_SELECTION_H_ +#ifndef NETSURF_DESKTOP_SELECTION_H_ +#define NETSURF_DESKTOP_SELECTION_H_ #include #include "netsurf/mouse.h" @@ -37,13 +37,7 @@ typedef enum { DRAG_END } seln_drag_state; - -/* this structure should be treated as opaque outside selection.c - (it's defined here to accelerate selection_defined(s) for reduced - impact on redraw code) */ - -struct selection -{ +struct selection { struct content *c; struct box *root; @@ -57,19 +51,19 @@ struct selection seln_drag_state drag_state; }; +/** + * determine if a selecion is active + */ +bool selection_active(struct selection *s); -/* bool selection_defined(struct selection *s); */ -#define selection_defined(s) ((s)->defined) - -/* bool selection_dragging(struct selection *s); */ -#define selection_dragging(s) ((s)->drag_state != DRAG_NONE) +bool selection_dragging(struct selection *s); -/* bool selection_dragging_start(struct selection *s); */ -#define selection_dragging_start(s) ((s)->drag_state == DRAG_START) +bool selection_dragging_start(struct selection *s); -/** Handles completion of a drag operation */ -/* void selection_drag_end(struct selection *s); */ -#define selection_drag_end(s) ((s)->drag_state = DRAG_NONE) +/** + * Handles completion of a drag operation + */ +void selection_drag_end(struct selection *s); /** * Creates a new selection object associated with a browser window. @@ -129,8 +123,9 @@ void selection_reinit(struct selection *s); * \param s selection object * \param redraw true iff the previously selected region of the browser * window should be redrawn + * \return true if selection was cleared false if not */ -void selection_clear(struct selection *s, bool redraw); +bool selection_clear(struct selection *s, bool redraw); /** * Selects all the text within the box subtree controlled by -- cgit v1.2.3