On Jan 30, 2009, at 11:28 AM, Paul McNett wrote:

> This patch confuses me. Why should we allow for self._records to be  
> anything *but* a
> dDataSet?

        I wasn't worried about "allowing"; I was handling cases where it  
wasn't. Since the purpose of UnfilteredDataSet is to allow the  
cancel() to operate on all records, filtered or not, if _records is a  
plain tuple, by definition there are no filtered records.

> Also, there's a typo: self_records should be self._records...


        Guess my testing didn't hit that case and trigger an exception. Fixed.


-- Ed Leafe




_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: 
http://leafe.com/archives/byMID/[email protected]

Reply via email to