[Patch] Custom RTTI PropInfo for custom types

hnb · 2016-06-30 09:43:47

Hi!

I have idea for custom RTTI PropInfo for custom types. Patch attached:

(moved below)

Example usage:

type
  TDouble_Precision2 = type Double;

  TSQLPropInfoRTTIDouble_Precision2 = class(TSQLPropInfoRTTIDouble)
  public
    procedure GetValueVar(Instance: TObject; ToSQL: boolean;
      var result: RawUTF8; wasSQLString: PBoolean); override;
  end;

implementation

procedure TSQLPropInfoRTTIDouble_Precision2.GetValueVar(Instance: TObject; ToSQL: boolean;
  var result: RawUTF8; wasSQLString: PBoolean);
var
  D: Double;
begin
  if wasSQLString<>nil then
    wasSQLString^ := false;
  Result := FormatFloat('0.00', PropInfo.GetExtendedValue(Instance));
end;

begin
  RegisterSQLPropInfo(TypeInfo(TDouble_Precision2), TSQLPropInfoRTTIDouble_Precision2);
end.

Last edited by hnb (2016-07-06 06:17:23)

hnb · 2016-07-06 06:19:39

Ping. Attached patch has no performance issue for existing code base... That feature has critical meaning for my projects.

Corrected patch: https://drive.google.com/file/d/0B4PZhd … sp=sharing

hnb · 2016-07-11 07:07:44

Any feedback ? Can I commit my changes to fossil?

ab · 2016-07-11 12:48:36

Please do it.

ab · 2016-07-12 16:42:55

I've updated the implementation, to use a class procedure TSQLPropInfoRTTI.RegisterTypeInfo() for ORM custom types.
Makes more sense to me than a global function.
See http://synopse.info/fossil/info/76b39f90090f5

hnb · 2016-07-13 11:04:10

Looks much better -,-

ab · 2016-07-13 16:22:54

But it is just some new looking appearance.
Idea is from you and your patch!

mORMot Open Source

#1 2016-06-30 09:43:47

[Patch] Custom RTTI PropInfo for custom types

#2 2016-07-06 06:19:39

Re: [Patch] Custom RTTI PropInfo for custom types

#3 2016-07-11 07:07:44

Re: [Patch] Custom RTTI PropInfo for custom types

#4 2016-07-11 12:48:36

Re: [Patch] Custom RTTI PropInfo for custom types

#5 2016-07-12 16:42:55

Re: [Patch] Custom RTTI PropInfo for custom types

#6 2016-07-13 11:04:10

Re: [Patch] Custom RTTI PropInfo for custom types

#7 2016-07-13 16:22:54

Re: [Patch] Custom RTTI PropInfo for custom types

Board footer