Re: pgadmin3 GTK

From: efesar <efesar(at)nmia(dot)com>
To: pgadmin-hackers(at)postgresql(dot)org, Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
Subject: Re: pgadmin3 GTK
Date: 2003-04-08 21:05:30
Message-ID: NGBBKFMOILMAGDABPFEGOEJBECAA.efesar@nmia.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers


> Please do not use
> for (int si=0 ; ...)
> Instead, declare int si; separately. You never know what the compiler
> really does from that.

Good to know. I'll make sure that's in future code.

> I found several OnXXX() methods, that are probably used in EventMaps, I
> didn't check this. Please make sure they have the correct arguments. If
> they don't have a sufficient argument, you won't be able to run a
> Release-Build. Unfortunately, the compiler will not warn you about this.

As far as I know, I've used the correct event maps. Hopefully I'm using the
correct number of arguments on everything, but I'm not 100% sure (sometimes
the docs are not explicit, and the only way to know for sure is by seeing
examples). As I'm coding I'll double check them.

One concept I'm having trouble understanding is the WXUNUSED macro.

-Keith

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Tom Lane 2003-04-09 04:14:16 Re: pg_get_viewdef 7.4 et al
Previous Message Dave Page 2003-04-08 19:09:05 Re: binary frx file diff