Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Wed, 19 Aug 2015 23:23:45 +0000 (UTC)
From: "Sebb (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.12857354.1440026566000.112809.1440026625928@Atlassian.JIRA>
In-Reply-To: <JIRA.12857354.1440026566000@Atlassian.JIRA>
References: <JIRA.12857354.1440026566000@Atlassian.JIRA>
 <JIRA.12857354.1440026566501@arcas>
Subject: [jira] [Created] (BCEL-249) Check for max Short seems wrong
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Sebb created BCEL-249:
-------------------------

             Summary: Check for max Short seems wrong
                 Key: BCEL-249
                 URL: https://issues.apache.org/jira/browse/BCEL-249
             Project: Commons BCEL
          Issue Type: Bug
            Reporter: Sebb


There are some checks for the max  value that will fit in a short.
These are of the form:

{code}
if (Math.abs(value) >= 32767)
{code}

This seems wrong, as Short.MAX_VALUE = 32767, so the check should be

{code}
if (Math.abs(value) > 32767)
{code}

There's a further problem, which is that Short.MIN_VALUE = -32768.
This will be disallowed, unless the condition is written as:

{code}
if ((value >= -32768) && (value <= 32767))
{code}

This is how the ctor generic.PUSH.PUSH(ConstantPoolGen cp, int value) does the check.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)