Re: [pgadmin-hackers] Fixed some compiler warnings

2014-02-04 Thread Akshay Joshi
Hi Dave I have removed the check like "if (i > 0 || i ==0)" from many places except one. I have tried to fix compiler warnings either by modifying the logic or putting some other checks. Attached is the modified patch, please review it. On Mon, Feb 3, 2014 at 3:13 PM, Akshay Joshi wrote: > > >

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

2014-02-04 Thread Shane StClair
Thanks J.F. I was thinking that being able to pipe the SQL through an external tool would increase flexibility for users. Everyone likes their SQL a certain way (look at the options on http://poorsql.com/ for instance). Options like these could be built into pgadmin but that seems like a lot of wor

[pgadmin-hackers] Fwd: [pgadmin-support] Format SQL (external tool)

2014-02-04 Thread Shane StClair
[Moved to pgadmin-hackers] Original message: Saturday, February 1, 2014, 2:17:20 AM, you wrote: Hi list, Just suggesting an idea. It would be really nice to be able to configure an external command to format SQL in pgAdmin. After configuring the external tool path in prefs, user hits a key combo