Skip site navigation (1) Skip section navigation (2)

Re: s/xpm/png/g

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: s/xpm/png/g
Date: 2011-03-07 17:18:17
Message-ID: AANLkTime-JkAUJAkWee3R-2A1+dsfJpLUAPODr_bPVva@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On Mon, Mar 7, 2011 at 2:55 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> On Mon, Mar 7, 2011 at 14:55, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> Hi,
>>
>> The (large) patch at
>> http://developer.pgadmin.org/~dpage/embedded_images_infra-v8.diff
>> replaces all the XPM images in pgAdmin with PNG ones. This offers us
>> two major advantages:
>>
>> 1) XPM images aren't supported by many graphics tools, and almost
>> always require manual editing to fix the internal naming anyway.
>>
>> 2) XPM images don't support alpha transparency.
>>
>> The patch is a little complex, as it's not straightforward to embed
>> PNG images at build time. Here's what it does:
>>
>> - Adds a new project, png2c, on which the pgAdmin project is dependent.
>> - Adds a new build rule for .png files, which will cause them to be
>> pre-processed with png2c, to create .pngc files which are C source
>> code, containing the PNG data and some accessor functions and macros.
>
> Didn't we get rid of the beerware license in postgresql, to make
> corporate lawyers happy? Are you sure it's a good idea to introduce it
> to pgadmin here?

I don't see why not. It doesn't *require* us to do anything. It just
says *if we think* it's worth it, we *can* buy him a beer.

>> - #include's the .pngc files in place of the XPMs.
>> - Updates the rest of the code to use the new accessor macros created
>> in the .pngc files:
>>
>>    filename_png_img - represents a wxImage*
>>    filename_png_bmp - represents a wxBitmap*
>>    filename_png_ico - represents a wxIcon*
>
> Just to make it easier to browse and search, I'd use a prefix for that
> stuf.. E.g. img_filename_... So you can do a prefix search and find
> all references. Or maybe embed_filename_png_xyz.

Maybe. I can't get that excited about it, as we've lived with _xpm for
the last 9 years.

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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

In response to

Responses

pgadmin-hackers by date

Next:From: Thom BrownDate: 2011-03-07 17:36:12
Subject: Re: [DOCS] Remove name as valid parameter for catalog functions
Previous:From: Tom LaneDate: 2011-03-07 15:19:00
Subject: Re: [DOCS] Remove name as valid parameter for catalog functions

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group