W dniu 14 wrzeÅ›nia 2010 23:25 użytkownik Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> napisaÅ‚:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On Sep 14, 2010, at 10:41 AM, Marcin Åšwiderski wrote:<br>
<br>
> I'm used to using named parameters object as temporary created in function's parameters list, just like I did in AnalysisContext.cpp . Such approach is clearer IMO, because it shows exactly what I want to achieve: pass parameters to function, and not create an object named B and pass it to function.<br>

><br>
> But that's just my opinion and I can remove those setters if it will better fit the whole project.<br>
<br>
</div>I think it's an elegant design, but in this case it adds 17 lines of code for the method definitions just to simplify two callsites where the CFG is constructed. Â In that regards it seems like overkill. Â If this was an API that would be used in many places (and the object copies could be elided), I'd be right there with you. Â I'd rather just keep the BuildOptions object simple.</blockquote>
</div>Fixed