[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