Re: Couple of minor fixes

From: "Magnus Hagander" <mha(at)sollentuna(dot)net>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Couple of minor fixes
Date: 2006-04-09 19:51:10
Message-ID: 6BCB9D8A16AC4241919521715F4D8BCEA0F8F5@algol.sollentuna.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

> >>> I'm also seeing a lot of signed/unsigned mismatch, but
> IIRC it was
> >>> said at some point that we don't care about those.
> >>
> >> In HEAD? They're all cleaned up according to the compilers I use.
>
> > Yup.
> > More digging then I guess. Probably a #define that's messe dup
> > somewhere.
>
> Maybe the issue is in Windows-specific code? I have no idea
> what the warning status is for that stuff. In all the stuff
> I regularly compile, there are only a couple of warnings and
> they are flex's fault (so there's not much we can do about
> 'em). I think it's good to get rid of noise warnings as much
> as possible --- otherwise they can make you miss important ones.

Probably. I'll see what I can find. It could also be my build being
broken (I've hacked up the build system pretty bad right now), but I
think I used to see this before as well. Will get back to you on this
;-)

//Magnus

Browse pgsql-patches by date

  From Date Subject
Next Message Gregory Maxwell 2006-04-09 20:36:54 Re: Support Parallel Query Execution in Executor
Previous Message Tom Lane 2006-04-09 19:48:56 Re: Couple of minor fixes