[PATCH] D56099: [Sanitizer] Add fstab api to FreeBSD/NetBSD

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 27 13:16:42 PST 2018


vitalybuka added a comment.

LGTM



================
Comment at: test/sanitizer_common/TestCases/Posix/getfsent.cc:28
+
+  printf("First entry: device block '%s', mounted with '%s'\n", fentry->fs_spec, fentry->fs_mntops);
+
----------------
krytarowski wrote:
> Line too long.
I'd expect if clang-format long lines was not disabled for tests

```
printf("First entry: device block '%s', mounted with '%s'\n",
       fentry->fs_spec, fentry->fs_mntops);
```


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56099/new/

https://reviews.llvm.org/D56099





More information about the llvm-commits mailing list