From 6c185b74bcb67555a8476ad9973a761db543c490 Mon Sep 17 00:00:00 2001 From: Kris De Gussem Date: Fri, 20 Jan 2012 21:17:56 +0100 Subject: cppcheck tells us 'line 154: "Possible null pointer dereference: _stream - otherwise it is redundant to check if _stream is null at line 147" Cannot check code on linux, but when you should get here _stream should be not NULL (if I interpreted correctly) (bzr r10915) --- src/extension/internal/latex-text-renderer.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/extension/internal/latex-text-renderer.cpp b/src/extension/internal/latex-text-renderer.cpp index fae10a0fe..12b42a78c 100644 --- a/src/extension/internal/latex-text-renderer.cpp +++ b/src/extension/internal/latex-text-renderer.cpp @@ -144,12 +144,10 @@ LaTeXTextRenderer::setTargetFile(gchar const *filename) { g_free(filename_ext); } - if (_stream) { - /* fixme: this is kinda icky */ + /* fixme: this is kinda icky */ #if !defined(_WIN32) && !defined(__WIN32__) - (void) signal(SIGPIPE, SIG_IGN); + (void) signal(SIGPIPE, SIG_IGN); #endif - } fprintf(_stream, "%%%% Creator: Inkscape %s, www.inkscape.org\n", PACKAGE_STRING); fprintf(_stream, "%%%% PDF/EPS/PS + LaTeX output extension by Johan Engelen, 2010\n"); -- cgit v1.2.3