[Lldb-commits] [PATCH] D49949: Add missing boundary checks to variable completion.

2018-08-01 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. One trivial fix to the test case. Repository: rL LLVM https://reviews.llvm.org/D49949 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D49949: Add missing boundary checks to variable completion.

2018-07-27 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL338177: Add missing boundary checks to variable completion. (authored by teemperor, committed by ). Herald added a subscri

[Lldb-commits] [PATCH] D49949: Add missing boundary checks to variable completion.

2018-07-27 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor updated this revision to Diff 157803. teemperor added a comment. - Fixed formatting. https://reviews.llvm.org/D49949 Files: packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py packages/Python/lldbsuite/test/functionalities/completion/main.cpp source/Symbo

[Lldb-commits] [PATCH] D49949: Add missing boundary checks to variable completion.

2018-07-27 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. Stopgap patch to at least stop all the crashes I get from this code. https://reviews.llvm.org/D49949 Files: packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py packages/Python/lldbsuite/test/functionalities/completion/main.cpp sourc