I really loved your patch because it actually made dash smaller!
On Sun, Jul 05, 2009 at 12:45:49AM +0200, Stefan Potyra wrote:
>
> +static void
> +readcmd_handle_line(char *line, char **ap, size_t len)
> +{
> + struct arglist arglist;
> + struct strlist *sl;
> + char *s, *backup;
> +
Hi,
Am Sunday 28 June 2009 06:19:02 schrieb Herbert Xu:
> Awesome! The patch looks pretty good in general. There are just
> a few minor issues that we need to fix.
Thanks for your thorough review! Revised patch is attached.
>
> > +static void
> > +readcmd_handle_line(char *line, char **ap)
> >
Hi,
sorry for the long delay.
attached is a patch, which uses ifsbreakup for field splitting. This one is a
little bit more invasive, as I've basically rewritten readcmd, and then tried
to realign it back with the old version.
Cheers,
Stefan.
--- dash-0.5.5.1.orig/src/expand.c
+++ dash-0.
On Wed, Sep 19, 2007 at 09:29:25AM +, Gerrit Pape wrote:
> When trying to split fields by tabs, dash doesn't honour multiple tabs
> between fields as whitespace (at least that's how I interpret [1],
> please correct me if I'm wrong).
>
> [1]:
> http://www.opengroup.org/onlinepubs/95399/ut
When trying to split fields by tabs, dash doesn't honour multiple tabs
between fields as whitespace (at least that's how I interpret [1],
please correct me if I'm wrong).
[1]:
http://www.opengroup.org/onlinepubs/95399/utilities/xcu_chap02.html#tag_02_06_05
Patch is from Stefan Potyra <[EMAIL
5 matches
Mail list logo
