On 12/5/05, Junio C Hamano <junkio@cox.net> wrote: > >> This attempts to clean up the way various compatibility > >> functions are defined and used. > >> --- a/compat/mmap.c > >> +++ b/compat/mmap.c > > > > I still think that compat functions should stand alone. > > Especially if it does not costs us much (or even less than that). > Sorry, you lost me. What do you mean by "standing alone"? Do not depend nor use anything from the calling code (do not take anything from git). If it has to serve as compatibility wrapper to mmap, it better should be mmap(2): return MAP_FAILED if error, do not exit process, do no output. I wouldn't expect open(2) to print "File not found", would you? Besides, MAP_FAILED is perfectly handled in calling layers. If they even print out errno, the case is completely covered and simplier handled. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Tue Dec 06 18:15:52 2005
This archive was generated by hypermail 2.1.8 : 2005-12-06 18:15:59 EST