[haiku-commits] Re: haiku: hrev44837 - src/apps/deskbar

  • From: Axel DÃrfler <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 14 Nov 2012 20:33:21 +0100

On 11/14/2012 08:02 PM, jscipione@xxxxxxxxx wrote:
   Might fix #9151 although since I can't reproduce the bug here I have
   no way of knowing.

Not just unlikely, however...

+       if (fAppListMenuItem != NULL) {
+               RemoveItem(static_cast<BMenuItem*>(fAppListMenuItem));

... why should the cast be necessary in the first place? fAppListMenuItem is a TBarMenuTitle which directly subclasses BMenuItem without any special tricks involved.

Bye,
   Axel.


Other related posts: