diff options
| author | Kris De Gussem <kris.degussem@gmail.com> | 2012-01-20 20:12:10 +0000 |
|---|---|---|
| committer | Kris <Kris.De.Gussem@hotmail.com> | 2012-01-20 20:12:10 +0000 |
| commit | 621e07d7f94bc806b259d0d662b1d3ecb9fa4778 (patch) | |
| tree | d717c6da12076bd0b2de9181ce2c439cd1e8dc5f /src | |
| parent | fix memleak (diff) | |
| download | inkscape-621e07d7f94bc806b259d0d662b1d3ecb9fa4778.tar.gz inkscape-621e07d7f94bc806b259d0d662b1d3ecb9fa4778.zip | |
cppcheck
tells us: "Possible null pointer dereference: _stream - otherwise it is redundant to check if _stream is null at line 105"
Cannot check code on linux, but when you should get here _stream should be not NULL (if I interpreted correctly)
(bzr r10913)
Diffstat (limited to 'src')
| -rw-r--r-- | src/extension/internal/latex-pstricks.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/extension/internal/latex-pstricks.cpp b/src/extension/internal/latex-pstricks.cpp index 1d36d9456..4b8d926f9 100644 --- a/src/extension/internal/latex-pstricks.cpp +++ b/src/extension/internal/latex-pstricks.cpp @@ -101,12 +101,10 @@ unsigned int PrintLatex::begin (Inkscape::Extension::Print *mod, SPDocument *doc g_free(local_fn); - 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 - } res = fprintf(_stream, "%%LaTeX with PSTricks extensions\n"); /* flush this to test output stream as early as possible */ |