<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 4, 2013 at 1:45 PM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul.Robinson@am.sony.com" target="_blank">Paul.Robinson@am.sony.com</a>></span> wrote:<br>

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




<div>
<div style="direction:ltr;font-size:10pt;font-family:Tahoma">
<p> </p>
<div style="font-size:16px;font-family:Times New Roman">
<div>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><div>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div>
<div style="FONT-FAMILY:Tahoma;DIRECTION:ltr;FONT-SIZE:10pt">
<div style="FONT-FAMILY:Times New Roman;FONT-SIZE:16px">
<div>
<div dir="ltr">
<div>
<div>
<div>> b) Instead of inlining all the code for CollectRecordStaticVars how about outlining it into a couple of static functions?</div>
<div> </div>
</div>
<div>Hmm, yeah the body of that loop is basically "if static-member do this; else if field do that;" so factoring out those bits seems reasonable.  (The lambda part, which I didn't touch, probably also could be factored out for similar reasons, but that might
 exceed the scope of what I'm doing with this patch.  Let me know.)</div>
<div>
<div> </div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
<div><br>
</div>
<div>> Yes please, thanks.</div>
<div> </div>
</div><div>It turns out to be infeasible to make these static functions, because they use private data and methods from class CGDebugInfo.  Do you want me to do the outlining as new private methods, or just leave everything inline within CollectRecordFields?  I will
 say that the refactoring does make things look cleaner.</div>
<div><br></div></div></div></div></div></div></div></div></blockquote><div><br></div><div style>New private methods will be fine, thanks. :)</div><div style><br></div><div style>-eric</div><div> </div></div><br></div></div>