[Lldb-commits] [PATCH] D35305: Remove uint32_t assignment operator from Status

2017-11-01 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317093: Remove uint32_t assignment operator from Status (authored by labath). Repository: rL LLVM https://reviews.llvm.org/D35305 Files: lldb/trunk/include/lldb/Utility/Status.h lldb/trunk/source/

[Lldb-commits] [PATCH] D35305: Remove uint32_t assignment operator from Status

2017-10-31 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. Sure, this seems fine to me. Not sure why this was introduced, the svn history has become hard to follow. But this seems overly specific for an integer assignment operator. https://revi

[Lldb-commits] [PATCH] D35305: Remove uint32_t assignment operator from Status

2017-07-12 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. It is not presently used, and it's quite dangerous to use -- it assumes the integer is an osx kern_return_t, but very few of the integers we have lying around are mach kernel error codes. The error can still be used to a mach error using a slightly longer (but more ex