aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKelsey Judson <kelseyjudson@protonmail.com>2018-01-01 19:52:40 +1300
committerKelsey Judson <kelseyjudson@protonmail.com>2018-01-01 19:52:40 +1300
commit664b3ee8b72ef815227b9f1fd76ee0ca72f3704a (patch)
tree9ebcaa0e2a2b86d487392c17159df35067cdb8f3
parent6557d0ee823418d3e78a4133fd5d5af9b5319bca (diff)
downloadvis-664b3ee8b72ef815227b9f1fd76ee0ca72f3704a.tar.gz
vis-664b3ee8b72ef815227b9f1fd76ee0ca72f3704a.tar.xz
vis-open: fix for absolute and non-existent paths
When the shell cannot find any matching files, the glob is not expanded, and vis-open will return the absolute path of the current working directory (because dirname outputs '.'), followed by the filename, followed by a literal '*'. This commit checks that the final path actually exists, and if not, exits with status 1. It also uses text_object_longword for the range to match, so that absolute paths are accepted, and replaced properly (else it only works back to the first '/').
-rw-r--r--lua/plugins/complete-filename.lua5
-rw-r--r--vis-lua.c9
-rwxr-xr-xvis-open5
3 files changed, 17 insertions, 2 deletions
diff --git a/lua/plugins/complete-filename.lua b/lua/plugins/complete-filename.lua
index cb5b361..98e8ec3 100644
--- a/lua/plugins/complete-filename.lua
+++ b/lua/plugins/complete-filename.lua
@@ -6,7 +6,7 @@ vis:map(vis.modes.INSERT, "<C-x><C-f>", function()
local pos = win.selection.pos
if not pos then return end
-- TODO do something clever here
- local range = file:text_object_word(pos > 0 and pos-1 or pos);
+ local range = file:text_object_longword(pos > 0 and pos-1 or pos);
if not range then return end
if range.finish > pos then range.finish = pos end
if range.start == range.finish then return end
@@ -29,7 +29,8 @@ vis:map(vis.modes.INSERT, "<C-x><C-o>", function()
local file = win.file
local pos = win.selection.pos
if not pos then return end
- local range = file:text_object_word(pos > 0 and pos-1 or pos);
+ -- TODO do something clever here
+ local range = file:text_object_longword(pos > 0 and pos-1 or pos);
if not range then return end
if range.finish > pos then range.finish = pos end
local prefix = file:content(range)
diff --git a/vis-lua.c b/vis-lua.c
index 3c998e7..124e380 100644
--- a/vis-lua.c
+++ b/vis-lua.c
@@ -2234,6 +2234,14 @@ static int file_mark_get(lua_State *L) {
* @treturn Range range the range
*/
+/***
+ * WORD text object.
+ *
+ * @function text_object_longword
+ * @tparam int pos the position which must be part of the word
+ * @treturn Range range the range
+ */
+
static int file_text_object(lua_State *L) {
Filerange range = text_range_empty();
File *file = obj_ref_check(L, 1, VIS_LUA_TYPE_FILE);
@@ -2703,6 +2711,7 @@ void vis_lua_init(Vis *vis) {
const char *name;
} textobjects[] = {
{ VIS_TEXTOBJECT_INNER_WORD, "text_object_word" },
+ { VIS_TEXTOBJECT_INNER_LONGWORD, "text_object_longword" },
};
for (size_t i = 0; i < LENGTH(textobjects); i++) {
diff --git a/vis-open b/vis-open
index 997aa49..cb81b04 100755
--- a/vis-open
+++ b/vis-open
@@ -40,6 +40,11 @@ if [ $# -eq 1 -a "$ALLOW_AUTO_SELECT" = 1 ]; then
# If there were globs on the command-line, they've expanded to
# a single item, so we can just process it.
+ # If the file or directory does not exist, abort.
+ if [ ! -e "$1" ]; then
+ exit 1
+ fi
+
if [ -d "$1" ]; then
# Recurse and show the contents of the named directory,
# We pass -f to force the next iteration to present the