[Synfig-devl] [PATCH] fix sequence point warning in state_brush.cpp

2015-06-22 Thread Olaf Hering
gcc 4.8 in openSUSE 13.1 warns about this code: states/state_brush.cpp:824:44: warning: operation on 'hue' may be undefined [-Wsequence-point] : hue = 60.0 * (((r - g)/(diff))+4.0); Remove early assignment and adjust indention to clearify logical code flow. Signed-off-by: Olaf Hering o

[Synfig-devl] [PATCH] lyr_freetype: fix crash if FcFontList returns empty FcFontSet

2015-06-22 Thread Olaf Hering
If FcFontList returns an empty FcFontSet synfig will crash because fs-fonts is a NULL pointer. Avoid that crash by checking the number of returned fonts. Signed-off-by: Olaf Hering o...@aepfle.de --- src/modules/lyr_freetype/lyr_freetype.cpp |2 +- 1 file changed, 1 insertion(+), 1 deletion

[Synfig-devl] [PATCH] synfig.pc: include all use libraries in devel package

2015-06-22 Thread Olaf Hering
If synfigstudio.spec has just BuildRequires: pkgconfig(synfig) some required libs will not be installed. In my case pkgconfig(Magick++). As a result linking failed. Put more libs into CONFIG_DEPS to automatically install them if synfig-devel is installed. Signed-off-by: Olaf Hering o

[Synfig-devl] [PATCH] configure.ac: do not touch g++ option -g

2015-06-22 Thread Olaf Hering
and other -g variants. Signed-off-by: Olaf Hering o...@aepfle.de --- m4/subs.m4 |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: synfig-1.0/m4/subs.m4 === --- synfig-1.0.orig/m4/subs.m4 +++ synfig-1.0/m4/subs.m4

Re: [Synfig-devl] [PATCH] configure.ac: do not touch g++ option -g

2015-06-22 Thread Olaf Hering
On Mon, Jun 22, Konstantin Dmitriev wrote: 2015-06-22 15:32 GMT+06:00 Olaf Hering o...@aepfle.de: CFLAGS and CXXFLAGS may contain other valid options than just -g. Leave anything related to debuginfo as instructed by the user to avoid syntax errors. Prepend the private debug knobs