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. |