<div>Kostis,</div><div><br></div>Crap, I missed the export_type.  Interesting I was able to run Dialyzer on stdlib with no problems (related to this change).  Anyways, I'll fix this and resubmit.<div><br></div><div>-Ryan<br>
<br><div class="gmail_quote">On Mon, May 23, 2011 at 11:11 AM, Kostis Sagonas <span dir="ltr"><<a href="mailto:kostis@cs.ntua.gr">kostis@cs.ntua.gr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Ryan Zezeski wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
I found it odd the there is no canonical proplist() type defined so this is my attempt at adding one.<br>
<br></div>
git fetch git://<a href="http://github.com/rzezeski/otp.git" target="_blank">github.com/rzezeski/otp.git</a> <<a href="http://github.com/rzezeski/otp.git" target="_blank">http://github.com/rzezeski/otp.git</a>> add_proplist_type<br>

</blockquote>
<br>
Ryan,<br>
<br>
Your attempt is not OK.<br>
<br>
For it to be, the proplist() type should be exported from the proplists module; i.e. added in the export_type declaration of that module. So, if your patch is to be accepted and included in OTP, minimally this needs to be fixed.<br>

<br>
I thought that since property() is an exported type from the proplists module, there is no need to define proplist() since its definition would be simple. But I think I was wrong: Your patch definitely shows that the current situation with various different "definitions" of proplist() is not optimal...<br>

<br>
Thanks for noticing this.<br><font color="#888888">
<br>
Kostis<br>
</font></blockquote></div><br></div>