The patch from 2-1/2 years ago for changing copy-unsafe-links to follow unsafe links on the destination side also included essentially this patch. When I looked at it, however, I asked why in the world is unsafe_symlink() doing strdup() in the first place. I think you could get rid of the calls to strdup() and the new local variables and possibly do a couple casts inside the function instead.
- Dave On Wed, Jan 15, 2003 at 12:49:44PM -0500, [EMAIL PROTECTED] wrote to rsync-cvs: > Re: CVS update: rsync > > Date: Wed Jan 15 17:49:44 2003 > Author: wayned > > Update of /data/cvs/rsync > In directory dp.samba.org:/tmp/cvs-serv3194 > > Modified Files: > proto.h util.c > Log Message: > Make unsafe_symlink() take const args so that we don't get any > compiler warnings when calling it with a const char *. > > > Revisions: > proto.h 1.150 => 1.151 > http://www.samba.org/cgi-bin/cvsweb/rsync/proto.h?r1=1.150&r2=1.151 > util.c 1.117 => 1.118 > http://www.samba.org/cgi-bin/cvsweb/rsync/util.c?r1=1.117&r2=1.118 > _______________________________________________ > rsync-cvs mailing list > [EMAIL PROTECTED] > http://lists.samba.org/mailman/listinfo/rsync-cvs -- To unsubscribe or change options: http://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.tuxedo.org/~esr/faqs/smart-questions.html