Re: [PATCH] - libstore: Add prefix @ to device to pass master device

[Samuel Thibault]

Hello,

Damien Zammit, le dim. 24 nov. 2019 12:33:59 +1100, a ecrit:
> So for example, with this patch I can call:
> 
>       settrans /dev/wd0s2 /hurd/storeio -T typed 
> part:2:device:@/dev/rump:/dev/wd0

Any comment on the syntax, anybody?

>      {
> +      /* Parse @master:/dev/hello */
> +      master = strdup (name);
> +      copy = strdup (name);
> +      pos = strchr (copy, ':');
> +      *pos = '\0';

Instead of duplicating all the string and truncating it, you can strchr
inside name, and strndup that.

You also don't need to strdup for "rest", you can just use rest =
name+something.

Thanks!
Samuel