| From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
|---|---|
| To: | Thom Brown <thom(at)linux(dot)com> |
| Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: [FEATURE] OIDs check box to enable OIDs |
| Date: | 2011-07-03 21:18:44 |
| Message-ID: | 1309727924.2061.0.camel@laptop |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
On Sun, 2011-07-03 at 12:34 +0100, Thom Brown wrote:
> 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.
>
Commited, and pushed. Thank you.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Guillaume Lelarge | 2011-07-03 21:39:16 | pgAdmin III commit: Use the new syntax for sequence when possible |
| Previous Message | Guillaume Lelarge | 2011-07-03 21:18:41 | pgAdmin III commit: Wrap SQL lines for ALTER commands |