[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