[Lldb-commits] [PATCH] D32732: "target variable" not showing all global variable if we print any global variable before execution starts

[vignesh balu via lldb-commits]

Any suggestions ?
thanks,
vbalu

On Tue, May 16, 2017 at 12:51 PM, vignesh balu via Phabricator <
reviews at reviews.llvm.org> wrote:

> vbalu added a comment.
>
> In https://reviews.llvm.org/D32732#753348, @jingham wrote:
>
> > That looks like the right way to do it.  What was your thinking behind
> returning null rather than the partial list of variables already parsed if
> can_create is false?  That doesn't seem like what somebody who is bothering
> to pass in can_create false intends.
>
>
> The issue is use of m_variables when it is partially Parsed. when a
> function looking of a global variable count it just call the
> GetVariableList with "can_create=false" which will return the partially
> parsed m_variabe leads to wrong count.
> "can_create" avoid the recursion while parsing the variables and to get
> the parsed global variable list to the callee. So i thought returning NULL
> should be wise instead of sending partially parsed list.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D32732
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170522/8f9fc410/attachment.html>