Hi,
Some comments about the patch: - The TLS stuff is ok and it can be checked in.
I'll see if I can separate things..
- I think the patch should follow the existing mono coding conventions, ie. no pascal-casing, space before and after /* */, before-and after operators etc.
Yes, I know that the identifiers are not really C like. (Somehow I thought I should follow the C# coding convention.) Of course I'll rework all identifiers that are used in mini.c/mini-amd64.c (etc.) before checking in. The other files (in the mini/contopt dir) I think can be converted 'on demand', tough -personally- I would like to see them not to be changed too much..
I'll try to review the more important parts of the patch just haven't got the time to do it yet. Also, Paolo needs to review it too before a patch this big can be checked in.
No problem. I'm currently doing the paper work here, so I've got plenty to do..
BTW: I just realized that I added all my 'custom backend' files too; Of course these files (ma64*) are NOT needed and won't be checked in.
Willi
_______________________________________________ Mono-devel-list mailing list Mono-devel-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-devel-list
