This revision was automatically updated to reflect the committed changes.
Closed by commit rG52712d3ff7a2: Re-land "get rid of
PythonInteger::GetInteger()" (authored by lawrence_danna).
Changed prior to commit:
https://reviews.llvm.org/D78462?vs=259916&id=262908#toc
Repository:
rG LLVM Githu
omjavaid accepted this revision.
omjavaid added a comment.
This revision is now accepted and ready to land.
Thanks for getting this fixed. Looks good now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
lawrence_danna added a comment.
@omjavaid ok to re-land?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
___
lldb-commits mailing list
lldb-commits@lists.llvm.o
lawrence_danna updated this revision to Diff 259916.
lawrence_danna added a comment.
rebased
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
lldb/bindings/python/python-typemaps.swig
lldb/binding
labath accepted this revision.
labath added a comment.
Still looks good.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
___
lldb-commits mailing list
lldb-comm
lawrence_danna updated this revision to Diff 259700.
lawrence_danna added a comment.
rebased
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
lldb/bindings/python/python-typemaps.swig
lldb/binding
lawrence_danna added a comment.
@omjavaid sorry I didn't catch that on pre-submit testing.
`PyLong_AsLongLong` and friends do not automatically convert on python2 like
they do on python3. It's fixed now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llv
lawrence_danna updated this revision to Diff 259659.
lawrence_danna added a comment.
fix python2 projblems
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
lldb/bindings/python/python-typemaps.swig
omjavaid requested changes to this revision.
omjavaid added a comment.
This revision now requires changes to proceed.
I have temporarily reverted this change to turn buildbot green.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://r
This revision was not accepted when it landed; it landed in state "Needs
Revision".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG478619cf9a24: Revert "get rid of
PythonInteger::GetInteger()" (authored by omjavaid).
Changed prior to commit:
https:
omjavaid reopened this revision.
omjavaid added a comment.
This revision is now accepted and ready to land.
This causes multiple test failures on LLDB AArch64 Linux buildbot.
http://lab.llvm.org:8011/builders/lldb-aarch64-ubuntu/builds/3695
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LA
This revision was automatically updated to reflect the committed changes.
Closed by commit rG737521217295: get rid of PythonInteger::GetInteger()
(authored by lawrence_danna).
Changed prior to commit:
https://reviews.llvm.org/D78462?vs=259131&id=259134#toc
Repository:
rG LLVM Github Monorepo
lawrence_danna updated this revision to Diff 259131.
lawrence_danna marked 4 inline comments as done.
lawrence_danna added a comment.
wrap long line
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
lawrence_danna added inline comments.
Comment at: lldb/bindings/python/python-wrapper.swig:585-597
+llvm::Expected result = pfunc.Call(PythonString(child_name));
-if (!result.IsAllocated())
-return UINT32_MAX;
+long long retval = unwrapOrSetPythonException(A
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
Looks good. I'll leave it up to you to consider whether the `PySys_WriteStderr`
thingy is a good idea.
Comment at: lldb/bindings/python/python-typemaps.swig:72
+ PythonObje
lawrence_danna updated this revision to Diff 258831.
lawrence_danna added a comment.
fix
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
lldb/bindings/python/python-typemaps.swig
lldb/bindings/py
lawrence_danna added inline comments.
Comment at:
lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp:3153-3159
+ long long py_return = unwrapOrSetPythonException(
+ As(implementor.CallMethod(callee_name)));
// if it fails, print the error but oth
lawrence_danna updated this revision to Diff 258810.
lawrence_danna marked 4 inline comments as done.
lawrence_danna added a comment.
review fixes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78462/new/
https://reviews.llvm.org/D78462
Files:
ll
labath added inline comments.
Comment at:
lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp:55
+return obj.takeError();
+ return obj.get().AsUnsignedLongLong();
+}
`obj->AsUnsignedLongLong()` ?
Comment at:
lldb/source/Pl
lawrence_danna created this revision.
lawrence_danna added reviewers: labath, jasonmolenda, JDevlieghere, vadimcn.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
One small step in my long running quest to improve python exception handling in
LLDB. Replace GetInteger() whi
20 matches
Mail list logo
