Re: NSTableView update basics
Re: NSTableView update basics
- Subject: Re: NSTableView update basics
- From: Erik Stainsby <email@hidden>
- Date: Fri, 09 Mar 2012 12:19:09 -0800
A poorly thought out attempt to return the count or zero (I was writing a lot of perl the day I coded this).
I recall reading that there is a race condition when *view*-based tables are loaded from nib. This does not obtain in my current case: this is a cell based table.
I removed the errant || 0 and the table is behaving correctly. Thanks for proofing my code.
Erik Stainsby
email@hidden
-------------------------------------
Consistently place constants on the LHS of an expression: you cannot accidentally assign when you meant to compare.
On 2012-03-09, at 1:38 AM, Graham Cox wrote:
>
> On 09/03/2012, at 4:42 PM, Erik Stainsby wrote:
>
>> - (NSInteger) numberOfRowsInTableView:(NSTableView *)tableView {
>> return [tableData count] || 0;
>> }
>
>
> What is the intention here? You are doing a boolean OR with the value zero, which will promote (in fact demote) the return type to a BOOL. Whatever the number of items, it will only ever return YES, or 1.
>
> --Graham
>
_______________________________________________
Cocoa-dev mailing list (email@hidden)
Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden