To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=86443
User mmeeks changed the following:
What |Old value |New value
================================================================================
Ever confirmed| |1
--------------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
--------------------------------------------------------------------------------
------- Additional comments from [email protected] Mon Jan 5 10:18:27
+0000 2009 -------
Great catch - unfortunately there are a number of other places doing just the
same thing; eg. in the parent class:
svtabbx.cxx also has two instances of this construct:
sal_uInt16 nColumnCount = GetColumnCount();
sal_Int32 nRow = _nPos / nColumnCount;
Perhaps a better question is -- why is there a header there with no columns in
it ? it seems to me that returning 0 from GetColumnCount is prolly closer to the
bug than adding lots of special cases for this around the place eg.
void SvHeaderTabListBox::RecalculateAccessibleChildren()
{
if ( !m_aAccessibleChildren.empty() )
{
sal_uInt32 nCount = ( GetRowCount() + 1 ) * GetColumnCount();
if ( m_aAccessibleChildren.size() < nCount )
m_aAccessibleChildren.resize( nCount );
is going to horribly malfunction with 0 GetColumnCount as well - ditto many of
the multiplies in there.
---------------------------------------------------------------------
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]
