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
erik.stain...@roaringsky.ca
-------------------------------------
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 (Cocoa-dev@lists.apple.com)
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:
https://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com
This email sent to arch...@mail-archive.com
