[PATCH 0/4] Small updates to wmaker's Info panel

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Following Rodolfo's proposed patch to add some information in the info dialog of WindowMaker, I propose the following 4 patches. The first patch is actually the change from Rodolfo with the feedback from Alexey implemented, it should (if git

[PATCH 2/4] InfoPanel: Minor fixes for what must be translated and what does not need to

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr --- src/dialog.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/dialog.c b/src/dialog.c index 3ff0214..c761f15 100644 --- a/src/dialog.c +++ b/src/dialog.c @@ -1257,7 +1257,8 @@ void wShowInfoPanel(WScreen * scr)

[PATCH 3/4] InfoPanel: Use a more user-friendly separator to display supported image formats

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr --- src/dialog.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/dialog.c b/src/dialog.c index c761f15..6f1950f 100644 --- a/src/dialog.c +++ b/src/dialog.c @@ -1118,6 +1118,7 @@ void wShowInfoPanel(WScreen * scr)

[PATCH 4/4] InfoPanel: Added display of memory fragmentation information

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr This information is displayed only when debug mode was enabled to avoid confusing users. --- src/dialog.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/src/dialog.c b/src/dialog.c index 6f1950f..2c2e5d3

[PATCH 1/4] InfoPanel: Added info about XRandR in the info dialog

2013-05-10 Thread Christophe
From: Rodolfo García Peñas (kix) k...@kix.es This patch includes info about XRandR extension in the info panel dialog. If wmaker was compiled with xrandr support, then the dialog show the XRandR info. The info includes if the X-Server supports or not XRandR (wmaker could be compiled with XRandR

[repo.or.cz] wmaker-crm.git branch next updated: wmaker-0.95.4-141-gf29fba2

2013-05-10 Thread crmafra
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project wmaker-crm.git. The branch, next has been updated via f29fba2c8392ccd88f83c1cf6482d144a2651d23 (commit) via

[PATCH 0/8] Some minor changes here and there

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Hello, Some small updates to clean things: - 01 to 03: deprecated stuff removal - 04 to 08: coding standards Best regards, Christophe. README | 10 --- WINGs/WINGs/WINGs.h | 21 +- WINGs/WINGs/WUtil.h | 21 ++

[PATCH 1/8] Readme: Removed reference to the tutorial

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr This page does not exist anymore, and may have been outdated anyway as it appears to be like 10 years old contribution. --- README |6 -- 1 file changed, 6 deletions(-) diff --git a/README b/README index 67c6f9e..12f1b9b 100644 ---

[PATCH 2/8] Readme: Removed reference to 'contrib' directory

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr This directory have been removed long time ago (in 2010 by Tamas TEVESZ, in commit c960ae622e48477b11cfbd6b2f6b98add3dca242) so we should not document it. --- README |4 1 file changed, 4 deletions(-) diff --git a/README b/README index

[PATCH 4/8] Configure: Changed OS dependant check to use Autoconf's macro

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr The macro provided by Autoconf will generate a similar code but with better portability, and the source file is more consistent. --- configure.ac | 41 + 1 file changed, 9 insertions(+), 32 deletions(-)

[PATCH 5/8] Changed WM_OSDEP to use Conditional instead of Substitution

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr The automake documentation states that using substitution inside the list of SOURCES will not work and calls for not doing it. The use of 'EXTRA_xxx' made things look like they worked but is probably not enough for corner cases. This patches

[PATCH 6/8] WPrefs: Added a few comments in header file

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Nothing really interresting, just separators to group definitions by theme, using the same syntax that can be found in other headers. --- WPrefs.app/WPrefs.h |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 7/8] Added proper legal notice at beginning of header files

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Some header were missing the notice; used git blame to find the original date and author (without guarantee. --- WINGs/WINGs/WINGs.h | 21 - WINGs/WINGs/WUtil.h | 21 + WINGs/wconfig.h | 19

[PATCH 8/8] Added missing include guards in a few headers

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Considering the number of headers we have, it is a good idea to avoid possible problems. For details, you may read: http://en.wikipedia.org/wiki/Include_guard All headers should be ok now. --- src/geomview.h |8 src/placement.h |

[repo.or.cz] wmaker-crm.git branch next updated: wmaker-0.95.4-149-g7c32044

2013-05-10 Thread crmafra
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project wmaker-crm.git. The branch, next has been updated via 7c320447fbabf36527dbd2c3405cb18ab0bfbb19 (commit) via

Re: [PATCH 5/8] Changed WM_OSDEP to use Conditional instead of Substitution

2013-05-10 Thread Alexey I. Froloff
On Fri, May 10, 2013 at 06:35:42PM +0200, Christophe wrote: -EXTRA_wmaker_SOURCES = osdep_bsd.c osdep_darwin.c osdep_linux.c osdep_stub.c +if WM_OSDEP_BSD +wmaker_SOURCES += osdep_bsd.c +endif +if WM_OSDEP_DARWIN +wmaker_SOURCES += osdep_darwin.c +endif +if WM_OSDEP_LINUX +wmaker_SOURCES

Re : Re: [PATCH 5/8] Changed WM_OSDEP to use Conditional instead of Substitution

2013-05-10 Thread Christophe
- Alexey I. Froloff ra...@raorn.name a écrit : On Fri, May 10, 2013 at 06:35:42PM +0200, Christophe wrote: -EXTRA_wmaker_SOURCES = osdep_bsd.c osdep_darwin.c osdep_linux.c osdep_stub.c +if WM_OSDEP_BSD +wmaker_SOURCES += osdep_bsd.c +endif +if WM_OSDEP_DARWIN +wmaker_SOURCES

[PATCH 00/11] More miscelleanous fixes here and there

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Hello, Yet another serie of patch with a lot of unrelated little things: - 01: memory fragmentation - 02, 11: configure improvment - 03, 04: portability improvment - 05, 07 to 10: bug risk reduction - 06: bug fix Best regards, Christophe.

[PATCH 01/11] Removed temporary allocation to build a path that is actually a constant

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr This allocation was certainly participating to memory fragmentation. --- WPrefs.app/Menu.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/WPrefs.app/Menu.c b/WPrefs.app/Menu.c index 7780df1..85639b0 100644 ---

[PATCH 04/11] Unified usage of the 'inline' attribute for functions

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Autoconf provides the necessary stuff to detect if inline keyword is supported, and to detect special syntaxes, so let's use this and remove the multiple local definitions, this makes code simpler. --- WINGs/hashtable.c | 10 +++---

[PATCH 05/11] WPrefs: Move declaration of function into the common header

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr It is a bad idea to declare the prototype of an external function in a file as it won't allow the compiler to cross-check it. --- WPrefs.app/MouseSettings.c |1 - WPrefs.app/WPrefs.h|4 2 files changed, 4 insertions(+), 1

[PATCH 06/11] Fixed improper variables definition in header file

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr Some header were creating variable, this is a bad practice which is likely to not behave as expected. This creates one distinct variable in each object file that used the header, and: - on well behaved compiler, this ends up in a link error (see

[PATCH 07/11] WRaster: Changed equality/inequality comparison on floating point number

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr The equality comparison (a == b) is known to be a dangerous trap when floating-point arithmetics are involved. This patch changes all the cases which try to do this to a safer check. --- wrlib/png.c|2 +- wrlib/rotate.c | 21

[PATCH 08/11] WINGs: Changed the minimum internal knob size of WScroller

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr The original code allowed to have 0.0, but this can generate division by zero in WScrollView. As a value of 0.0 is not realistic anyway, use a minimum constant instead. --- WINGs/wscroller.c | 13 +++-- 1 file changed, 11 insertions(+), 2

[PATCH 10/11] wmaker: Removed equality comparison on floating point number

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr The equality comparison (a == b) is known to be a dangerous trap when floating-point arithmetics are involved. In the current case the offending operation can be done with integers directly. --- src/misc.c | 26 +++--- 1 file

[PATCH 11/11] Configure: Added some -Wxxx checks when debug mode is enabled

2013-05-10 Thread Christophe
From: Christophe CURIS christophe.cu...@free.fr We probably don't want our users to have to endure them, so they are enabled only when Debug is activated, because they tend to help keeping the code safe. --- configure.ac | 16 1 file changed, 16 insertions(+) diff --git

[repo.or.cz] wmaker-crm.git branch next updated: wmaker-0.95.4-160-gf282464

2013-05-10 Thread crmafra
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project wmaker-crm.git. The branch, next has been updated via f2824647f371986cefa4dba7b0e51a54d9798aed (commit) via