On Wednesday 11 January 2006 13:44, Morten Nilsen wrote:
> Brian Mattern wrote:
> > "conofig"? You should probably test before sending patches ;-)
> > I'll test/apply tonight (hopefully).
>
> I usually do.. but this was trivial patch for a program I never use.. I
> don't even think I have it compiled.. I just grepped for code that
> would/could need changes after my ecore_config argb changes..
> silly typo though..
>
> it turned out, argb/rgb wasn't really in use anywhere.. I hope that can
> change now.. no more need to inflict #aarrggbb formatting on apps..
A few things. First, what you changed is only the saving of colors so they can
be loaded up the next time. The config file size may go down slightly, but
there is no other benefit. Second, why didn't you use
ecore_config_rgb_{set,get}()? (then you wouldn't need to pass NULL or 255 to
the unused alpha params).
On a side note, anyone know why ecore_config uses argb instead of rgba?
(Handy?)
--
rephorm
-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems? Stop! Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
