From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Mads Jensen <mje(at)inducks(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Some trivial patch |
Date: | 2014-11-05 08:51:08 |
Message-ID: | CAG7mmow+5-ufbCHCMS-BJs+R-uMog-FOMs1bfdhcYyasfM_MGQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Mads,
As per our discussion, it is not improvement.
But - it is a unused inclusion of header, and usage of proper macro
provided by wxWidgets.
Committed the patch.
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com>
*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>
On Tue, Nov 4, 2014 at 9:53 AM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
wrote:
> On Mon, Nov 3, 2014 at 7:38 PM, Mads Jensen <mje(at)inducks(dot)org> wrote:
>
>> On 11/03/2014 02:42 PM, Ashesh Vashi wrote:
>> > Hi Mads,
>> >
>> > What is the exact improvement?
>> > I did not understand from the patch.
>> >
>> > Was the existing code not working properly?
>>
>> Yes, but aren't unnecessary imports (with misleading comments) deprecated?
>>
> Yes - it is.
> But - I was confused by the subject line, and missing (incomplete)
> description.
>
> --
>> Med Venlig Hilsen / Kind regards,
>> Mads Jensen
>>
>> Max Jerry Horovitz: "Unfortunately, in America, babies are not found in
>> cola cans. I asked my mother when I was four and she said they came from
>> eggs laid by rabbis. If you aren't Jewish, they're laid by Catholic
>> nuns. If you're an atheist, they're laid by dirty, lonely prostitutes."
>> -- Mary and Max (2009)
>>
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | J.F. Oster | 2014-11-05 14:25:10 | PATCH: Add missing nodes to graphical explain plan |
Previous Message | Ashesh Vashi | 2014-11-05 08:48:15 | pgAdmin III commit: Removed unused header file from the ctlSQLBox.cpp |