Re: [pgadmin-hackers] PATCH: Format SQL (external tool)

2015-07-27 Thread Ashesh Vashi
Hi J.F. Oster, Looks good to me. Committed to the repository on master branch. -- Thanks & Regards, Ashesh Vashi EnterpriseDB INDIA: Enterprise PostgreSQL Company *http://www.linkedin.com/in/asheshvashi* On Mon, Jul 20, 2

Re: [pgadmin-hackers] PATCH: Format SQL (external tool)

2015-07-20 Thread Dave Page
Have you got this Ashesh? On Sat, Jul 18, 2015 at 1:06 PM, J.F. Oster wrote: > Hi All, > > > I haven't received any feedback on last message... > > Attaching a new version of patch. Now it reads a value for timeout from > sysSettings, but I didn't add an input to Options form for reasons > expl

Re: [pgadmin-hackers] PATCH: Format SQL (external tool)

2015-06-26 Thread J.F. Oster
Hello Ashesh, Sorry for long absense. Are you sure we need that additional option for specifying timeout? How do you see it placed in the Options dialogue? For me it will look somewhat surplus and "overloading" the form. It will be needed to change extremely rarely. The only true case I can i