[haiku-commits] Re: haiku: hrev43279 - src/kits/interface src/apps/debugger/user_interface/gui/team_window headers/os/interface

  • From: Rene Gollent <anevilyak@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 17 Nov 2011 07:57:57 -0500

On Thu, Nov 17, 2011 at 4:29 AM, Stephan Aßmus <superstippi@xxxxxx> wrote:
> This addition is really strange. Why would you clamp the index? Instead of
> noticing a bug, the caller will query a wrong item. Crashing is fine in this
> case, if you absolutely don't want to crash (for what reason?), you can
> always return false. This at least achieves predictable behavior for out of
> bounds indices.

I simply did so because SetCollapsible() does so and I wanted to
remain consistent. Should I remove it across the board then?

Will make the naming adjustments shortly, thanks for the review!



Other related posts: