[haiku-commits] Re: haiku: hrev53368 - src/servers/media

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 13 Aug 2019 15:39:36 +0200

Hi,

On Tue, Aug 13, 2019 at 2:11 PM waddlesplash <waddlesplash@xxxxxxxxx> wrote:

On Tue, Aug 13, 2019, 5:02 AM Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
wrote:

Hi,

-                       if (msg->FindInt32("team", &id) == B_OK
+                       if (msg->FindInt32("be:team", &id) == B_OK


This is a funny and nice catch that just shows how fragile this BMessage
model is.


No. The SOME_APP_QUIT message has had its members named so since the Be
days.


Hmm, I've been referring the fact that something like that should not
happen.


This only shows you did not bother to test this code at all before you
committed it.


Not sure why you have this need to blame other people and pretend to be
better than others. However, realize that before my code there was not even
an attempt to clean resources.



Cannot possibly fix the other issue with buffers.


Considering you were wrong about this issue, I am more than willing to bet
you are wrong about this, too.


Yeah. Indeed, unlike you I am wrong sometimes. But in any case the problems
will be still there.

-- 
Saluti,
Dario

Other related posts: