<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jul 11, 2011, at 7:41 , Quanming Shi wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; ">In the functions , the block pointer is used as the ID of the specific ALUT block. I think this will be more convenient for users and don't need to search blocks or ID frequently. Nearly all of the API functions accept the block pointer as the first param.<span class="Apple-converted-space"> </span><br></span><br class="Apple-interchange-newline"></blockquote></div><div><br></div>Hi Quanming. I know I already reviewed it, but here is a note and a few things I didn't point out.<div><br></div><div>- You should make it clear that the rtems_mprot_alut structure is opaque, and though the user is returned the structure pointer the user doesn't have rights to poke around in it.</div><div>- Anyplace that you can you should declare that the pointed-to object is "const". For example, all "get_" functions should be const.</div><div>- The sizes should be "size_t" and not "unsigned int".</div><div>- Did you mean to leave a mixture of "_memp_" and "_mprot_" in the functions and data structures, or did you plan to change everything to "rtems_mprot"_? If you intended to have the two prefixes then what is the difference?</div><div><br></div><div><div>
<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><font class="Apple-style-span" size="3"><span class="Apple-style-span" style="font-size: 12px; ">Peter<br>-----------------<br>Peter Dufault<br>HD Associates, Inc. Software and System Engineering</span></font></div></span></span>
</div>
<br></div></body></html>