Hi Brian,

Thanks for the patches!

I haven’t look at the other patch yet, but about this one:

BT Templeton <b...@hcoop.net> writes:

> +#!/bin/sh

Can you add a copyright/license header here?

> +snarf_test ()
> +{
> +    local x
> +    x=$(snarf "$1")
> +    if [ x"$x" != x"$2" ]; then

‘local’ and ‘$(...)’ are Bash extensions.  Can you remove ‘local’ and
use backquotes instead of $(...)?

> +snarf_test "^^a^:^" "a;"
> +snarf_test "  ^  ^  b  ^  :  ^  " "b;"
> +snarf_test "c\n^^d^:^\ne" "d;"
> +snarf_test "f^^g^:^h" "g;"
> +snarf_test "^^i^:^j^^k^:^" "i;k;"

Which of these should work with the current guile-snarf, and which
should work only with yours?

Thanks!

Ludo’.


Reply via email to