Hi brian,
On Sun, 16 Jun 2019, brian m. carlson wrote:
> diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh
> index 8270de74be..11a6abca2e 100644
> --- a/t/test-lib-functions.sh
> +++ b/t/test-lib-functions.sh
> @@ -1314,6 +1314,12 @@ test_oid () {
> eval "printf '%s' \"\${$var}\""
> }
>
> +# Insert a slash into an object ID so it can be used to reference a location
> +# under ".git/objects". For example, "deadbeef..." becomes "de/adbeef..".
> +test_oid_to_path () {
> + echo "$1" | sed -e 's!^..!&/!'
> +}
I guess it does not *really* matter all that much, but this does spawn a
new process (and I think it actually spawns 4 on Windows, for reasons, and
spawning processes is super expensive on Windows).
We might actually want to think about using something like this instead
(which admittedly looks a bit like gobbledygook to the uninitiated, but it
definitely avoids any spawned process):
test_oid_to_path () {
echo "${1%${1#??}}/${1#??}"
}
Ciao,
Dscho
