<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<br>
<div class="moz-cite-prefix">Am 13.04.2015 um 12:02 schrieb Steve
Hildebrandt:<br>
</div>
<blockquote cite="mid:552B942F.1030005@web.de" type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<br>
<br>
<div class="moz-cite-prefix">Am 13.04.2015 um 09:38 schrieb
Alfred:<br>
</div>
<blockquote
cite="mid:em6a5cc76a-a5bb-4273-9f32-f2dc0a9b515e@alfsacer"
type="cite">
<style id="eMClientCss">blockquote.cite { margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc }
blockquote.cite2 {margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc; margin-top: 3px; padding-top: 0px; }
.plain pre, .plain tt { font-family: monospace; font-size: 100%; font-weight: normal; font-style: normal; }
a img { border: 0px; }body {font-family: Tahoma;font-size: 12pt;}
.plain pre, .plain tt {font-family: Tahoma;font-size: 12pt;}
</style>
<style></style>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div>Please do include me !</div>
<div> </div>
<div>I have included the rtti-patch (diff) based on the current
(today : 30566) trunk.</div>
<div> </div>
<div>Its nothing else than the patch-set from Steve, but with
one change:</div>
<div> </div>
<div>for k:=0 to def.paras.count-1 do<br>
begin<br>
para:=tparavarsym(def.paras[k]);<br>
{note from Alf:}<br>
{this was in the first patch set by Steve}<br>
{left out by Steve in the current patch}<br>
{?}<br>
{<br>
if (vo_is_hidden_para in para.varoptions) and not<br>
(vo_is_self in para.varoptions) then<br>
continue;<br>
}</div>
<div> </div>
<div> </div>
<div>Thanks, Alf.</div>
</blockquote>
This was made, when I started actually using the generated RTTI.<br>
It was kind of problematic to invoke a method, if you have no
information how to pass the self parameter along.<br>
<br>
mfg Steve<br>
</blockquote>
I think something got mixed up here. <br>
I got confused and didn't look into things properly, sorry.<br>
The Patches are alright as they are, but the unified diff seems
incorrect.<br>
<br>
+ if (vo_is_hidden_para in para.varoptions) and
not<br>
+ (vo_is_self in para.varoptions) then<br>
+ continue;<br>
<br>
should be included see "[PATCH 2/6] Added base for Reg/Off
calculation to paramgr"<br>
<br>
I've also attached a simple test.<br>
If you would rather have a unified diff, I can provide it.<br>
<br>
mfg Steve<br>
</body>
</html>