Re: [FEATURE] OIDs check box to enable OIDs

From: Thom Brown <thom(at)linux(dot)com>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [FEATURE] OIDs check box to enable OIDs
Date: 2011-07-03 11:34:39
Message-ID: CAA-aLv4H1Zzg=FaTOkWUNdELF3AW6cBW1899WmPQqDY=bAC1OA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On 2 July 2011 21:26, Thom Brown <thom(at)linux(dot)com> wrote:
> On 2 July 2011 20:20, Guillaume Lelarge <guillaume(at)lelarge(dot)info> wrote:
>> Great patch. It even helped me find a bug. Anyway, commited and pushed.
>> Thank you.
>
> Thanks for committing it. :)
>
>>> I personally prefer the ones with newlines, especially since I've been
>>> experimenting with tables having very long names, and some of the
>>> above statements require lots of horizontal scrolling on the SQL tab
>>> to see what the action will be.  Could we harmonise these?
>>
>> Yes, we should. Could you provide a patch?
>
> Sure, I'll take a look at it.

Okay, patch attached to wrap lines at various places.

--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment Content-Type Size
sql_line_wrap.patch text/x-patch 27.7 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Thom Brown 2011-07-03 14:10:55 [BUG] Index operator class + collation = bad SQL
Previous Message Guillaume Lelarge 2011-07-03 09:06:40 Re: pgAdmin III commit: Update CHANGELOG and make style.