<div class="gmail_quote">On Sat, Jan 19, 2013 at 1:25 AM, CRLF0710 <span dir="ltr"><<a href="mailto:crlf0710@gmail.com" target="_blank">crlf0710@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">Hi all.<div><br><div>#1 In src/types.lisp there is following code in function MEM-REF:<div><div>....</div><div>    (if (aggregatep parsed-type)</div><div>        (if (bare-struct-type-p parsed-type)</div></div>



<div>....</div><div>and its compiler macro has similiar code.</div><div>However, the gf bare-struct-type-p has no method for  foreign-array-type whose objects will obviously pass the aggregatep test. So it breaks some code (on ccl).</div>



<div>I suggest that case be added as a method too?(What should it (and even more user-defined types) return?) Or should the checking code be modified here? (As i understood, the whole bare struct mechanism is for compatibility...)</div>

</div></div></div></blockquote><div><br>I would prefer to leave the question of aggregate definition to someone else, as I am not clear on what constitutes an aggregate.  However, if you find code breaking, please post a "before and after" minimal example with the previous correct result and the current wrong result.  If you have a proposed method to fix the problem, can you add that to your branch and do a pull request?<br>

 <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>

<div><br></div><div>#2 And another thing, i've realized that in my last modification to the STRUCTURE-BY-VALUE-P function (the modification is already pushed to the repo) i should have tested to see if it is a bare struct type. Without the check it will put some deprecated style code into using libffi, which also works correctly but involves unnecessary libffi dependency. Should I fix that?</div>

</div></div></div></blockquote><div> </div><div>Yes, please.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>

<div><br></div><div><div>Regards,</div><div>Charles Lew</div><span class="HOEnZb"></span></div></div></div></div></blockquote><div><br>Liam <br></div></div><br>