[cfe-commits] [PATCH] RegionStore

[Ted Kremenek]

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>