zsh-users
 help / color / mirror / code / Atom feed
* Help with directory switching functions
@ 2015-11-24 10:32 Dominik Vogt
  2015-11-28 18:54 ` Bart Schaefer
  0 siblings, 1 reply; 3+ messages in thread
From: Dominik Vogt @ 2015-11-24 10:32 UTC (permalink / raw)
  To: Zsh Users

I'm trying to write a function that alternately switches between
two directories.  With "+-" I can repeatedly switch between the
current directory and the last one where I did anything (type any
command except switching directories).  There are still a few
rough edges that might be improved.

The function makes use of the chpwd and precmd functions.  chpwd
writes down the $HISTCMD of the last command that switched
directories, and precmd uses that information to mark the current
directory as one yuop might want to switch back later.  When such
a directory is left, chpwd finally stores its path in a variable
that is then used by "-+".

-- snip --
function _swapdir_chpwd () {
	local _PWD_A
	local _OLDPWD_A

	_CHPWD_N="$[HISTCMD + 1]"
	_PWD_A=$(readlink -f "$PWD")
	test x"$_PWD_A" = x"$_RECORDED_DIR_A"; then
		_RECORDED_DIR="$PWD"
	fi
	_OLDPWD_A=$(readlink -f "$OLDPWD")
	if test "$_IS_CWD_INTERESTING" = 1 -a ! x"$_PWD_A" = x"$_OLDPWD_A"; then
		# really leaving an interesting dir -> record it
		_RECORDED_DIR="$OLDPWD"
		_RECORDED_DIR_A="$_OLDPWD_A"
		_IS_CWD_INTERESTING="0"
	fi
}

function chpwd () {
	_swapdir_chpwd
}

function _swapdir_precmd () {
	test ! "$HISTCMD" = "$_CHPWD_N" && _IS_CWD_INTERESTING="1"
}

function precmd () {
	_swapdir_precmd
}

function _swapdir () {
	_IS_CWD_INTERESTING=1
	cd "$_RECORDED_DIR"
	_IS_CWD_INTERESTING=1
}

alias -- +-=_swapdir
-- snip --

Problems:

1. Command lines with multiple command cannot be handled properly
   because precmd is just called once for the line, not for each
   command.

  $ cd ~; ls; cd ..

2. Depending on $HISTCMD is a bit hacky.  What I'd really want to
   do is to look at the command being executed and decide
   individually which commands are "interesting" enough to warrant
   recording the current directory.

3. I'd prefer a shell builtin instead of "readlink".

Any suggestions for improvement are welcome.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Help with directory switching functions
  2015-11-24 10:32 Help with directory switching functions Dominik Vogt
@ 2015-11-28 18:54 ` Bart Schaefer
  2015-11-30 10:48   ` Dominik Vogt
  0 siblings, 1 reply; 3+ messages in thread
From: Bart Schaefer @ 2015-11-28 18:54 UTC (permalink / raw)
  To: vogt, Zsh Users

On Nov 24, 11:32am, Dominik Vogt wrote:
}
} I'm trying to write a function that alternately switches between
} two directories.  With "+-" I can repeatedly switch between the
} current directory and the last one where I did anything (type any
} command except switching directories).

[...]

} The function makes use of the chpwd and precmd functions.
} 
} Problems:
} 
} 1. Command lines with multiple command cannot be handled properly
}    because precmd is just called once for the line, not for each
}    command.

If you really need to sneak in before (or after) each command rather
than just each command line, read up on TRAPDEBUG, but see below.

} 2. Depending on $HISTCMD is a bit hacky.  What I'd really want to
}    do is to look at the command being executed and decide
}    individually which commands are "interesting" enough to warrant
}    recording the current directory.

If you use preexec instead of precmd you can examine the command line
instead of just examining the history event number.  I think that
would allow you to start with _IS_CWD_INTERESTING="0" and only set it
to 1 when something interesting happens (if I read correctly right
now, you assume interesting and then zero it in chpwd if nothing else
has happened yet).

Also note that preexec would happen before chpwd, so you'd need some
other corresponding logic changes.

} 3. I'd prefer a shell builtin instead of "readlink".

    _PWD_A="$PWD:A"

Or if not that, why not?

Also, possibly use "cd -q" in _swapdir to avoid running chpwd (the work
will already have been done in preexec, I think).

Finally, you might consider using the directory stack; push each of the
directories that seems interesting, and then you can pop back through
them, rather than only having the two most recent to swap between.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Help with directory switching functions
  2015-11-28 18:54 ` Bart Schaefer
@ 2015-11-30 10:48   ` Dominik Vogt
  0 siblings, 0 replies; 3+ messages in thread
From: Dominik Vogt @ 2015-11-30 10:48 UTC (permalink / raw)
  To: Zsh Users

On Sat, Nov 28, 2015 at 10:54:28AM -0800, Bart Schaefer wrote:
> On Nov 24, 11:32am, Dominik Vogt wrote:
> }
> } I'm trying to write a function that alternately switches between
> } two directories.  With "+-" I can repeatedly switch between the
> } current directory and the last one where I did anything (type any
> } command except switching directories).
> 
> [...]
> 
> } The function makes use of the chpwd and precmd functions.
> } 
> } Problems:
> } 
> } 1. Command lines with multiple command cannot be handled properly
> }    because precmd is just called once for the line, not for each
> }    command.
> 
> If you really need to sneak in before (or after) each command rather
> than just each command line, read up on TRAPDEBUG, but see below.

All right, I've tried that, and it gets executed way too often
(e.g. when pressing cursor-up that is bound to a history search
scripts).  On the other hand I don't want to do my own command
line parsing, so I'll just live with precmd for now.  Maybe the
situation never occurs anyway.

> } 2. Depending on $HISTCMD is a bit hacky.  What I'd really want to
> }    do is to look at the command being executed and decide
> }    individually which commands are "interesting" enough to warrant
> }    recording the current directory.
> 
> If you use preexec instead of precmd you can examine the command line
> instead of just examining the history event number.  I think that
> would allow you to start with _IS_CWD_INTERESTING="0" and only set it
> to 1 when something interesting happens (if I read correctly right
> now, you assume interesting and then zero it in chpwd if nothing else
> has happened yet).
> 
> Also note that preexec would happen before chpwd, so you'd need some
> other corresponding logic changes.
> 
> } 3. I'd prefer a shell builtin instead of "readlink".
> 
>     _PWD_A="$PWD:A"

Ah, cool, that's exactly what I was looking for.

> Also, possibly use "cd -q" in _swapdir to avoid running chpwd (the work
> will already have been done in preexec, I think).

Good point.  Had to rewrite my cd script first, though.

> Finally, you might consider using the directory stack; push each of the
> directories that seems interesting, and then you can pop back through
> them, rather than only having the two most recent to swap between.

I already use pushd and popd with autopushd all the time, so it's
not really an option.  I'll probably enhance the script to use a
stack because sometimes more history would be helpful.

Thanks a lot, Bart!

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-11-30 10:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-24 10:32 Help with directory switching functions Dominik Vogt
2015-11-28 18:54 ` Bart Schaefer
2015-11-30 10:48   ` Dominik Vogt

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).