Return-Path: 
 <dev-return-20569-apmail-poi-dev-archive=poi.apache.org@poi.apache.org>
Delivered-To: apmail-poi-dev-archive@www.apache.org
Received: (qmail 1985 invoked from network); 4 Feb 2011 16:44:30 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3)
  by minotaur.apache.org with SMTP; 4 Feb 2011 16:44:30 -0000
Received: (qmail 71983 invoked by uid 500); 4 Feb 2011 16:44:30 -0000
Delivered-To: apmail-poi-dev-archive@poi.apache.org
Received: (qmail 71000 invoked by uid 500); 4 Feb 2011 16:44:27 -0000
Mailing-List: contact dev-help@poi.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@poi.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@poi.apache.org>
List-Post: <mailto:dev@poi.apache.org>
List-Id: <dev.poi.apache.org>
Reply-To: "POI Developers List" <dev@poi.apache.org>
Delivered-To: mailing list dev@poi.apache.org
Received: (qmail 70988 invoked by uid 99); 4 Feb 2011 16:44:26 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 04 Feb 2011 16:44:26 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=5.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.22] (HELO thor.apache.org) (140.211.11.22)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 04 Feb 2011 16:44:25 +0000
Received: from thor.apache.org (localhost [127.0.0.1])
	by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id p14Gi3KF013368
	for <dev@poi.apache.org>; Fri, 4 Feb 2011 16:44:03 GMT
Received: (from daemon@localhost)
	by thor.apache.org (8.13.8+Sun/8.13.8/Submit) id p14Gi3Jw013367;
	Fri, 4 Feb 2011 11:44:03 -0500 (EST)
Date: Fri, 4 Feb 2011 11:44:03 -0500 (EST)
Message-Id: <201102041644.p14Gi3Jw013367@thor.apache.org>
From: bugzilla@apache.org
To: dev@poi.apache.org
Subject: DO NOT REPLY [Bug 50539] Error reading in a vml file with unmatched
 <br>
X-Bugzilla-Reason: AssignedTo
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: POI
X-Bugzilla-Component: XSSF
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: nick.burch@alfresco.com
X-Bugzilla-Status: RESOLVED
X-Bugzilla-Priority: P2
X-Bugzilla-Assigned-To: dev@poi.apache.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Changed-Fields: Status Resolution
In-Reply-To: <bug-50539-47293@https.issues.apache.org/bugzilla/>
References: <bug-50539-47293@https.issues.apache.org/bugzilla/>
X-Bugzilla-URL: https://issues.apache.org/bugzilla/
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
X-Virus-Checked: Checked by ClamAV on apache.org

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

Nick Burch <nick.burch@alfresco.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Nick Burch <nick.burch@alfresco.com> 2011-02-04 11:44:00 EST ---
Thanks for the patch and unit test

After some pondering, I decided to refactor the class and fix a bit to make it
clearer what is going on. I tweaked the test to try a wide range of buffer
sizes, to simulate all the different cases of the br in the overflow, and it
now passes all of them.

Fix committed in r1067217.

-- 
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