[cfe-commits] [PATCH] RegionStore
Ted Kremenek
kremenek at apple.com
Mon Oct 20 23:03:56 PDT 2008
On Oct 20, 2008, at 11:01 PM, Zhongxing Xu wrote:
>
>
> On Tue, Oct 21, 2008 at 1:58 PM, Ted Kremenek <kremenek at apple.com>
> wrote:
>
> On Oct 20, 2008, at 10:56 PM, Zhongxing Xu wrote:
>
> Although it's more wordy, perhaps we should rename GetSVal() to
> GetBinding() just to make the meaning of this method more clear.
>
> Renaming suggestion for the Store interface:
>
> Bind()/Retrieve()
> SetBinding()/GetBinding()
>
>
> Better ideas?
>
> Bind() and Retrieve() sound great (unless you think they are too
> ambiguous). Short and sweet. "Bind()" also follows more with
> functional programming; we're creating a new store with the binding
> added.
>
> Great. I'll do the renaming.
Sounds good!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20081020/4e2edd15/attachment.html>
More information about the cfe-commits
mailing list