Mailing-List: contact dev-help@poi.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "POI Developers List" <dev@poi.apache.org>
From: bugzilla@apache.org
To: dev@poi.apache.org
Subject: DO NOT REPLY [Bug 53101] FormulaEvaluator incorrectly evaluates sum
 over cell range > 255
Date: Thu, 19 Apr 2012 11:39:13 +0000
Message-ID: <bug-53101-47293-uqgTFn78Sq@https.issues.apache.org/bugzilla/>
In-Reply-To: <bug-53101-47293@https.issues.apache.org/bugzilla/>
References: <bug-53101-47293@https.issues.apache.org/bugzilla/>
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0

https://issues.apache.org/bugzilla/show_bug.cgi?id=53101

--- Comment #7 from Yegor Kozlov <yegor@dinom.ru> 2012-04-19 11:39:13 UTC ---
I thought about removing all &'s too. The purpose of these operators is to
guarantee that the the result fits in the .xls grid bounds (2^8 cols and 2^16
rows). We don't need them in general case, but I want research the problem and
make sure it is really so. 

Yegor

(In reply to comment #6)
> How about removing the &'s altogether?:
> 
> public ValueEval getRelativeValue(int relativeRowIndex, int
> relativeColumnIndex) {
> 
>     int rowIx = relativeRowIndex + getFirstRow();
>     int colIx = relativeColumnIndex + getFirstColumn();
> 
>     return _evaluator.getEvalForCell(rowIx, colIx);
> }
> 
> This passes all the test as well.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org