Bug 45252 - Improvement for Range.replaceText()
Summary: Improvement for Range.replaceText()
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: HWPF (show other bugs)
Version: 3.0-dev
Hardware: PC Mac OS X 10.4
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on: 45001
Blocks:
  Show dependency tree
 
Reported: 2008-06-22 23:13 UTC by N. Hira
Modified: 2008-06-28 11:58 UTC (History)
0 users



Attachments
Patch to Range.replaceText() to allow replace-all and updated unit test (4.91 KB, patch)
2008-06-22 23:13 UTC, N. Hira
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description N. Hira 2008-06-22 23:13:42 UTC
Created attachment 22157 [details]
Patch to Range.replaceText() to allow replace-all and updated unit test

The SVN version for Range.replaceText() does not permit the caller to replace all instances of a piece of text.  It also forces the caller to rebuild the Range after each call.

The supplied patch is an improved version of replaceText() that permits the caller to continue working with the Range, an overloaded version that replaces all instances, and an update to the relevant unit test.
Comment 1 Nick Burch 2008-06-28 11:58:30 UTC
Thanks for this fix and test, applied to svn trunk