Re: [PATCH v2 0/3] trace2 docs: note that BUG() sends an "error" event
- From
Jeff King <peff@peff.net>
- Date
- Apr 15, 2021, 10:10 UTC
- Message-ID
- <YHgQ+usXZseTclLT@coredump.intra.peff.net>
- In-Reply-To
- <cover-0.3-00000000000-20210413T090603Z-avarab@gmail.com>
On Tue, Apr 13, 2021 at 11:08:18AM +0200, Ævar Arnfjörð Bjarmason wrote:
Show 5 quoted lines
> A trivial update to the trace2 docs to fix an omission > with "BUG()" not being listed alongside error(), die() etc. > > v1 of this[1] added a non-fatal-but-logging bug() function, per the > discussion on v1 that's now gone.
All three look reasonable to me. I left a few comments, but you may want to just ignore them. ;)
-Peff