Hi,
nice patches :-) only style comments. It doesn't matter.
On 25/08/2012 11:32, Iain Patterson wrote:
> +static void allocFrameBorderPixel(Colormap colormap, char *color_name,
> unsigned long **pixel) {
> + XColor xcol;
IMO is better:
+static void allocFrameBorderPixel(Colormap colormap, char *color_name,
unsigned long **pixel)
+{
+ XColor xcol;
And
> --- a/src/framewin.h
> +++ b/src/framewin.h
> @@ -150,6 +150,8 @@ typedef struct WFrameWindow {
> int depth;
> Visual *visual;
> Colormap colormap;
> + unsigned long *border_pixel;
> + unsigned long *selected_border_pixel;
> } WFrameWindow;
--- a/src/framewin.h
+++ b/src/framewin.h
@@ -150,6 +150,8 @@ typedef struct WFrameWindow {
int depth;
Visual *visual;
Colormap colormap;
+ unsigned long *border_pixel, *selected_border_pixel;
} WFrameWindow;
Cheers.
kix
--
||// //\\// Rodolfo "kix" Garcia
||\\// //\\ http://www.kix.es/
--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.
