Skip site navigation (1) Skip section navigation (2)

Re: new patch for index DESC/NULLS FIRST/NULLS LAST

From: Quan Zongliang <quanzongliang(at)gmail(dot)com>
To: "Dave Page" <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: new patch for index DESC/NULLS FIRST/NULLS LAST
Date: 2008-11-22 10:43:20
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgadmin-hackers
> That code could use some formatting changes though
> We also try to leave empty lines between distinct blocks of code to
> aid readability.


> - lstColumns should always display the order and nulls values
> - The NULLs radio buttons should not have a shaded box around them.
> - I think the dialogue design would look nicer if the DESC and and
> NULLs options were on the same horizontal alignment, eg:
> DESC []        NULLs    O FIRST    O LAST


> - When creating an index with one column, using DESC, I clicked on the
> SQL tab and got an assert:

I don't get it in Windows XP.
Maybe, when there is default value, my code don't invoke wxListView's
SetItem() and left original value. Is it NULL in Mac?

Quan Zongliang
CIT Japan:
CIT China:

Attachment: supportdescv3.patch
Description: application/octet-stream (20.9 KB)

In response to


pgadmin-hackers by date

Next:From: Tomasz ZemanDate: 2008-11-23 11:30:36
Subject: pgAgent 2.0.0 compilation errors (types casting)
Previous:From: Guillaume LelargeDate: 2008-11-20 22:27:06
Subject: Re: Buglets in dlgFunction

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group