[haiku-commits] Re: haiku: hrev53346 - src/add-ons/kernel/drivers/network/emulex_oce/dev/oce

  • From: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 6 Aug 2019 13:27:03 +0200

Am 06/08/2019 um 10:07 schrieb Adrien Destugues:

- POCE_INTR_INFO ii = &sc->intrs[vector];

This doesn't really access anything yet, though; it just sets 'ii'
to something that would be out of bounds only if you'd actually try
to access it, which wasn't done yet. IOW it just pleases the tool.
Yes, but it makes things more readable for humans as well, I think.

Maybe I'm not human after all ;-) In any case, I don't mind this change at all.

I would just report this as false positive, though (if possible), and I don't think it's worth upstreaming it.


Other related posts: