[lld] r297850 - [ELF] - Linkerscript: make Dot public and remove getDot(). NFC.
    Rui Ueyama via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Mar 16 15:02:16 PDT 2017
    
    
  
I reverted the change since that was made based on a miscommunciation.
So if we have a setter and a getter that don't do anything particularly
useful such as
  T getFoo() { return Foo; }
  void setFoo(T Val) { Foo = Val; }
then it makes sense to remove them, but in this case we have a fairly
complicated `setDot` function, and we always set Dot via that function. So
getDot works as a guard to protect the direct access to that variable.
On Thu, Mar 16, 2017 at 6:46 AM, George Rimar <grimar at accesssoftek.com>
wrote:
> >I didn't mean to make it public, but protected.
>
>
> It was protected before. It is used from ScriptParser, so I think either
> we can have
>
> getDot() like before, or make Dot public. I was sure you meant that when
> suggested to remove getter.
>
>
> George.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170316/2a941930/attachment.html>
    
    
More information about the llvm-commits
mailing list