View Issue Details

IDProjectCategoryView StatusLast Update
0006495JEDI Code LibraryJclAnsiStringspublic2019-04-18 17:50
ReporterpaulocmorenoAssigned ToAHUser 
PrioritynormalSeverityfeatureReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in VersionVersion 2.5 (Subversion repository/Daily zips) 
Summary0006495: Add the virtual method SetUpdateState in TJclAnsiStrings class
DescriptionNowadays the class TJclAnsiStrings doesn't implement the method SetUpdateState and the child class TJclAnsiStringList doesn't have the events OnChange and OnChanging. Since the methods BeginUpdate and EndUpdate aren't virtual, it's not possible to implement these behaviors in child classes of TJclAnsiStrings.

If possible, I suggest that TJclAnsiStrings and TJclAnsiStringList implement these methods and events, allowing to use these classes as TStrings and TStringList replacements. If this it's not desired to JCL, I suggest that you change the methods BeginUpdate and EndUpdate to be virtual, allowing the child classes implement these behaviors.
TagsNo tags attached.
Fixed in GIT commit
Fixed in SVN revision
IDE versionAll

Activities

mh

2019-04-17 22:02

reporter   ~0021717

Pull request with an implementation issued:
https://github.com/project-jedi/jcl/pull/58

AHUser

2019-04-18 17:50

developer   ~0021720

Fixed in master branch.

Issue History

Date Modified Username Field Change
2016-02-02 14:06 paulocmoreno New Issue
2016-02-02 14:06 paulocmoreno IDE version => All
2019-04-17 22:02 mh Note Added: 0021717
2019-04-18 17:50 AHUser Note Added: 0021720
2019-04-18 17:50 AHUser Status new => resolved
2019-04-18 17:50 AHUser Fixed in Version => Version 2.5 (Subversion repository/Daily zips)
2019-04-18 17:50 AHUser Resolution open => fixed
2019-04-18 17:50 AHUser Assigned To => AHUser