Mantis Bugtracker
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0006579 [Squeak] Collections minor always 07-27-07 20:25 04-18-10 22:04
Reporter nicolas cellier View Status public  
Assigned To nicolas cellier
Priority normal Resolution fixed  
Status closed   Product Version 3.9
Summary 0006579: String>>#findLastOccuranceOfString:startingAt: isBuggedWhen: [substring first=substring last]
Description
assert: ('ababa' findLastOccuranceOfString: 'aba' startingAt: 1) = 3
Additional Information Current algorithm find aba at: 1, then restart search at: 4, thus never find 3...

minimal example:

assert: ('aaa' findLastOccuranceOfString: 'aa' startingAt: 1) = 2
Attached Files

- Relationships
related to 0006366closed andreas [BUG] WideString findString:startingAt: and other matchTable problems 

- Notes
(0013245 - 95 - 101 - 255 - 255 - 255 - 255)
nicolas cellier
08-22-09 22:08

Serge Stinckwich posted a fix in pharo at
http://code.google.com/p/pharo/issues/detail?id=1019 [^]
 
(0013246 - 176 - 182 - 510 - 510 - 510 - 510)
robert_krahn
08-23-09 00:23

Proposal for a fix: http://source.squeak.org/inbox/Collections-rkrk.117.mcz [^] (replacing subString size with 1)
Test: http://source.squeak.org/tests/CollectionsTests-rkrk.90.mcz [^]
 
(0013247 - 38 - 38 - 38 - 38 - 38 - 38)
robert_krahn
08-23-09 00:24

Oops, saw note #1 first after posting.
 
(0013256 - 201 - 216 - 216 - 216 - 216 - 216)
nicolas cellier
08-24-09 19:21

That does not much matter, your solutions are the same.
However note that Serge also renamed Occurance -> Occurence
The right way to do that is to deprecate the old selector after adding the new one.
 
(0013340 - 103 - 103 - 103 - 223 - 223 - 223)
nicolas cellier
10-05-09 01:36

Note that both serge and robert changes are in conflict with 0006366, and I had to merge the corrections.
 

- Issue History
Date Modified Username Field Change
07-27-07 20:25 nicolas cellier New Issue
08-22-09 22:08 nicolas cellier Note Added: 0013245
08-23-09 00:23 robert_krahn Note Added: 0013246
08-23-09 00:24 robert_krahn Note Added: 0013247
08-24-09 19:21 nicolas cellier Note Added: 0013256
10-04-09 20:39 nicolas cellier Status new => resolved
10-04-09 20:39 nicolas cellier Fixed in Version  => trunk
10-04-09 20:39 nicolas cellier Resolution open => fixed
10-04-09 20:39 nicolas cellier Assigned To  => nicolas cellier
10-05-09 01:36 nicolas cellier Note Added: 0013340
10-05-09 01:36 nicolas cellier Relationship added related to 0006366
04-18-10 22:04 andreas Status resolved => closed


Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
60 total queries executed.
43 unique queries executed.
Powered by Mantis Bugtracker