[Bug c++/42669] New: libcpp: five redundant tests

public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed

* [Bug c++/42669]  New: libcpp: five redundant tests
@ 2010-01-09  8:50 dcb314 at hotmail dot com
  2010-01-09  9:59 ` [Bug preprocessor/42669] " paolo dot carlini at oracle dot com
  0 siblings, 1 reply; 3+ messages in thread
From: dcb314 at hotmail dot com @ 2010-01-09  8:50 UTC (permalink / raw)
  To: gcc-bugs

I just ran the most excellent sourceforge tool "cppcheck" over the source
code of the GNU C compiler version 4.5 snapshot 20100107 and cppcheck said
lots of things. An extract is below.

[./libcpp/init.c:260]: (style) Redundant condition. It is safe to deallocate a
NULL pointer
[./libcpp/files.c:999]: (style) Redundant condition. It is safe to deallocate a
NULL pointer
[./libcpp/pch.c:725]: (style) Redundant condition. It is safe to deallocate a
NULL pointer
[./libcpp/pch.c:727]: (style) Redundant condition. It is safe to deallocate a
NULL pointer
[./libcpp/pch.c:729]: (style) Redundant condition. It is safe to deallocate a
NULL pointer

I've manually checked all of these and I agree with cppcheck -
the redundant conditions may be safely deleted.

-- 
           Summary: libcpp: five redundant tests
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dcb314 at hotmail dot com
  GCC host triplet: x86_64-suse-linux

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42669

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug preprocessor/42669] libcpp: five redundant tests
  2010-01-09  8:50 [Bug c++/42669] New: libcpp: five redundant tests dcb314 at hotmail dot com
@ 2010-01-09  9:59 ` paolo dot carlini at oracle dot com
  0 siblings, 0 replies; 3+ messages in thread
From: paolo dot carlini at oracle dot com @ 2010-01-09  9:59 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from paolo dot carlini at oracle dot com  2010-01-09 09:59 -------
Didn't I tell you already that these issues are *not* suited for Bugzilla,
because aren't user visible? Just mention the issue on gcc, or send a patch to
gcc-patches.


-- 

paolo dot carlini at oracle dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |preprocessor


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42669


^ permalink raw reply	[flat|nested] 3+ messages in thread

[parent not found: <bug-42669-4@http.gcc.gnu.org/bugzilla/>]

* [Bug preprocessor/42669] libcpp: five redundant tests
       [not found] <bug-42669-4@http.gcc.gnu.org/bugzilla/>
@ 2013-05-13  6:43 ` skannan at redhat dot com
  0 siblings, 0 replies; 3+ messages in thread
From: skannan at redhat dot com @ 2013-05-13  6:43 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42669

Shakthi Kannan <skannan at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |skannan at redhat dot com

--- Comment #2 from Shakthi Kannan <skannan at redhat dot com> ---
With cppcheck-1.59 on Fedora 18 (x86_64) and gcc-4.8.0, the "Redundant
condition" is not present.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-05-13  6:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-01-09  8:50 [Bug c++/42669] New: libcpp: five redundant tests dcb314 at hotmail dot com
2010-01-09  9:59 ` [Bug preprocessor/42669] " paolo dot carlini at oracle dot com
     [not found] <bug-42669-4@http.gcc.gnu.org/bugzilla/>
2013-05-13  6:43 ` skannan at redhat dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).