View Issue Details

IDProjectCategoryView StatusLast Update
0036276FPCRTLpublic2019-11-08 11:44
ReporterAlexey Tor.Assigned ToMichael Van Canneyt 
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionwon't fix 
Product Version3.3.1Product Build 
Target VersionFixed in Version 
Summary0036276: TStringList.AddCommaText .AddDelimitedText can be faster
DescriptionMethods create temporary TStrings object and uses it to parse delimited text.
Suggestion-- take my TATStringSeparator from
https://github.com/Alexey-T/ATSynEdit/blob/master/atsynedit/atstringproc.pas
(it is "record" to be fast).
You need only one method: GetItemStr.
It finds items in delimited text super fast.
TagsNo tags attached.
Fixed in Revision
FPCOldBugId
FPCTarget-
Attached Files

Activities

Michael Van Canneyt

2019-11-07 14:08

administrator   ~0119133

The method is not supposed to be fast.
It's a convenience method, I see no reason to pull in extra code to make it faster.
It was doubtful it would be included to begin with, and if it had contained a lot of extra code, I would not have added it.

Issue History

Date Modified Username Field Change
2019-11-07 13:05 Alexey Tor. New Issue
2019-11-07 14:08 Michael Van Canneyt Assigned To => Michael Van Canneyt
2019-11-07 14:08 Michael Van Canneyt Status new => resolved
2019-11-07 14:08 Michael Van Canneyt Resolution open => won't fix
2019-11-07 14:08 Michael Van Canneyt FPCTarget => -
2019-11-07 14:08 Michael Van Canneyt Note Added: 0119133
2019-11-08 11:44 Alexey Tor. Status resolved => closed