[Mono-dev] msvc patch for eglib

Atsushi Eno atsushi at ximian.com
Thu Sep 14 12:01:27 EDT 2006


Mmm, actually the EOL stuff is somehow my bad. I'll change them to LF.

Atsushi Eno

Miguel de Icaza wrote:
> Hello Michael,
> 
>     A few comments on the patch:
> 
> 	* The changes require ChangeLog entries that detail the changes
> 
> 	* The coding style for some routines like gdate.c should follow
> 	  the coding conventions used in the rest of the code.
> 
> 	* Am not sure that the changes with varargs will continue to 
> 	  work on Unix, could you test?
> 
> 	* There are a lot of whitespace or newline changes in your
> 	  patch which make it impossible for me to determine what
> 	  changed, please fix that.
> 
> 	* vasprintf: please attach the license that the implementation
> 	  in freedts uses, if it is not MIT X11 we can not use it.
> 
> Miguel.
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
> 




More information about the Mono-devel-list mailing list