Re: [PATCH] Add-rm-rf-as-examples-loadables-rm.c

[Cedric Blancher]

Some comments, since ksh93 already has the same (POSIX utilities as builtins):
1. All builtin code should be reentrant (you'll get to that point at
*least* with find(1) -exec, and at that point it will bite you into
your *ARSE*)
2. All builtin code must not use global variables or any global
resources "write" (see (1), same warning, bloody *arse* if you try)
3. Be aware of locale changes (also global resource)

Ced


On 31 October 2016 at 12:59, Tim Ruehsen <tim.ruehsen@gmx.de> wrote:
> Added the 'rm' command to examples/loadables.
>
> It accepts -r and -f.
> Enable it with 'enable -f <path-to-bash>/examples/loadables/rm rm'.
>
> Tested on wget configure script where it increases overall execution time by
> 6-7%.
>
> Tim



-- 
Cedric Blancher <cedric.blancher@gmail.com>
[https://plus.google.com/u/0/+CedricBlancher/]
Institute Pasteur