To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=63686





------- Additional comments from [EMAIL PROTECTED] Fri Apr  7 09:27:35 -0700 
2006 -------
The members list should never contain null pointer entries, so that check seems
unnecessary.

But operator== in the patch is wrong. It's currently used only for the
ScPivotItem in which the dialog transports its results, and the only items that
really have to compare as equal are a cloned item and its original, but with the
patch, even that isn't guaranteed (the iterator order might be different).

The unpredictable order is also a bit of a problem when saving files. By result,
the files are equivalent with any order of data-pilot-member elements, but it's
at least an aethetic issue, and possibly a problem for automated testing. So
far, we generally tried to avoid such cases.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to