<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 20, 2013 at 9:02 AM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1hm">   This is wrong and needs fixing, having one name for the public interface and a different name in the function table. You need to change the name in the function table to get() and DMCreateLocalToGlobalMapping_Composite() and friends to use the name DMGet…<br>
</div></blockquote><div><br></div><div style>That would imply that each DM* implementation would need to manage caching, which is a bit of boilerplate, but perhaps okay in the interest of consistency. Or do we start/reinforce a convention that some implementation Get methods are called at most once because the outer part manages caching?</div>
<div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1hm">
   Having this inconsistent naming already wasted my time, and your time responding to the email and everyone else's time in having to delete the email. I'd fix it but git workflow is too hard so you need to fix it. Consistency reduces complexity.</div>
</blockquote></div><div class="gmail_extra"><br></div><div class="gmail_extra" style>I don't mind making the change, but here's the process to get a new branch into 'next':</div><br>$ git checkout -b name/dmgetlocaltoglobalmapping master</div>
<div class="gmail_extra" style>change source, test, and commit</div><div class="gmail_extra" style>$ git checkout next</div><div class="gmail_extra" style>$ git merge name/dmgetlocaltoglobalmapping</div><div class="gmail_extra" style>
$ test</div><div class="gmail_extra" style>$ git push origin next name/dmgetlocaltoglobalmapping</div><div class="gmail_extra" style><br></div></div>