Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Data-Dumper CPAN distribution.

Report information
The Basics
Id: 95407
Status: new
Priority: 0/
Queue: Data-Dumper

People
Owner: Nobody in particular
Requestors: jhi [...] iki.fi
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: suspicious switch case in Dumper.xs (Coverity finding)
Date: Tue, 06 May 2014 08:17:37 -0400
To: bug-Data-Dumper [...] rt.cpan.org
From: Jarkko Hietaniemi <jhi [...] iki.fi>
Suspicious = case1 + code + case2, without either flow control (break, continue) or annotation (/* FALLTHROUGH */) between the code and case2. Rationale: true fallthrough cases are rather rare, much more often break is intended. From 5-second inspection I assume this is an actually intentional fall-through, so adding /* FALLTHROUGH */ would be the thing to do. 206 case '\'': CID 28989 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value '\\') is not terminated by a 'break' statement. 207 case '\\': 208 *d = '\\'; 209 ++d; ++ret; fallthrough: The above case falls through to this one. 210 default: