Hi,

On Mon, Nov 29, 2004 at 03:29:40PM +0100, Jim Meyering wrote:
> Paul Eggert <[EMAIL PROTECTED]> wrote:
> > After looking at rpl_chdir's code, one possible simplification
> > suggests itself: use an argument of type "char *" (not "char const
> > *"), and temporarily modify it by setting '/' to '\0'.
> 
> IMHO, it is bad practice to modify such parameters, even temporarily,

OK, if you don't want the modify the real parameter, you can copy it, and
then freely modify the copy.

(I have to admit I haven't read your code, so I hope this advice is not
too silly...  ;-)

Regards,
        Stepan


_______________________________________________
Bug-coreutils mailing list
[EMAIL PROTECTED]
http://lists.gnu.org/mailman/listinfo/bug-coreutils

Reply via email to