[haiku-commits] Re: r37332 - haiku/trunk/src/apps/diskusage

  • From: Wim van der Meer <wpjvandermeer@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 2 Jul 2010 12:26:46 +0900

On Fri, Jul 2, 2010 at 12:22 PM, Rene Gollent <anevilyak@xxxxxxxxx> wrote:
> On Thu, Jul 1, 2010 at 11:16 PM,  <wpjvandermeer@xxxxxxxxx> wrote:
>> -                       status_t        Invoke()
>> +                       status_t        Invoke(BMessage* mesage = NULL)
>>                                                        { return 
>> BMenuItem::Invoke(); }
>
> Shouldn't that be: return BMenuItem::Invoke(message); ? Otherwise in
> the case when Invoke is called with an actual BMessage parameter on
> such an item, it will be ignored,
>
> Regards,
>
> Rene
>

This is a local class and is never called with a BMessage. Strictly
speaking you are correct.

-- 
Wim van der Meer
\/\///\/\

Other related posts: