On Sun, 2008-03-02 at 13:17 -0600, Raphael Geissert wrote: > Attached is a .diff which adds the following checks:
Thanks. Just a couple of quick comments: > '(?:^|\s+)unalias\s+-a' => q<unalias -a>, isn't a bashism. [http://www.opengroup.org/onlinepubs/009695399/utilities/unalias.html] > '(?:^|\s+)local\s+[^=;)}]+=' => q<local foo=bar>, > > Policy requires 'local' to be implemented, but local -r foo, local foo=bar, > and such are extras. No problem with this, although it'll probably get implemented as '(?:^|\s+)local\s+\w+=' ["a.b" isn't a valid variable name, even in bash] Adam -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
