search

kryslt / KControls

Free components for Delphi and Lazarus, this is the main repository maintained by the original author.
BSD 3-Clause Clear License
58 stars 32 forks source link

Change from variable (F*Class) in TKCustomGrid to virtual functions #15

Closed atlant2011 closed 4 years ago

atlant2011 commented 4 years ago
  1. Small old fix for D7. Incomplete Int64 type support in D7
  2. Change from variable (FClass) in TKCustomGrid to virtual functions returning "Class". And drop unneed(after apply this patch) functions RealizeCellClass, RealizeColClass,RealizeRowClass
kryslt commented 4 years ago

No way to delete protected or public methods. At least keep empty (dummy) methods. Did you test with all the demos? Need test at least in your Delphi and latest Lazarus.

atlant2011 commented 4 years ago

My option speeds up the code a little bit (you don 't have to re-create the objects). And the implementation is also simplified.

No way to delete protected or public methods.

You can try to leave the empty ones. It is necessary to check.

Did you test with all the demos?

Currently not fully tested (only latest Lazarus, Delphi - not installed). I 'll test and let you know.

atlant2011 commented 4 years ago

Sorry, this patch is bad for classic grid with different cell type. I'm test only *dbgrid (with my private prog) with column ones type. Drop this patch.

kryslt commented 4 years ago

Dropped as asked.