Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/sanitizer_common/sanitizer_mac.cc | ||
---|---|---|
341 ↗ | (On Diff #25407) | We don't use C89 declaration style, so please do: |
345 ↗ | (On Diff #25407) | Is it really can happen? Or it is more like "this should never happen"? |
347 ↗ | (On Diff #25407) | s/0U/0/ please |
LGTM with two nits.
lib/sanitizer_common/sanitizer_mac.cc | ||
---|---|---|
47 ↗ | (On Diff #25502) | While at it, can you please sort this block of includes alphabetically? |
343 ↗ | (On Diff #25502) | NB: currently task_info() can't possibly return anything but KERN_SUCCESS, so it's perfectly fine to die. |
345 ↗ | (On Diff #25502) | Nit: spare newline. |