[Mono-dev] Compiler warnings

Geoff Norton gnorton at novell.com
Thu Jul 23 23:24:49 EDT 2009

On 23-Jul-09, at 11:02 PM, Christian Hergert wrote:

> Hello,
> In an effort to get more familiar with some of the code-base, I went  
> through and fixed some of the pesky compiler warnings for the  
> runtime.  Attached is a patch for said warnings.
> If anyone has suggestions on how to better fix the warnings, please  
> send them my way and I'll adjust the patch as needed.

#1: Having tons of

is sucky, localize it into mono/utils/somewhere-logical.h and include  
it around

#2: lots of:
  	FILE *fp;

impedes readability (for me) just for 1 platform, not sure its worth it.

As for the rest, it looks sane, but I'll let the runtime guys weigh in.


More information about the Mono-devel-list mailing list