<br><font size=2 face="sans-serif">Joerg,</font>
<br>
<br><font size=2 face="sans-serif">Your fixes seem absolutely correct.
&nbsp;We welcome having bugs pointed out, especially in lesser used areas
- it is a big bonus when you also fix the bug.</font>
<br>
<br><font size=2 face="sans-serif">I have committed changes to CVS - thank
you.</font>
<br>
<br><font size=2 face="sans-serif">John Forrest</font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=40%><font size=1 face="sans-serif"><b>&quot;Joerg Herbers&quot;
&lt;Joerg.Herbers@inform-ac.com&gt;</b> </font>
<br><font size=1 face="sans-serif">Sent by: coin-discuss-bounces@list.coin-or.org</font>
<p><font size=1 face="sans-serif">12/19/2005 05:28 AM</font>
<table border>
<tr valign=top>
<td bgcolor=white>
<div align=center><font size=1 face="sans-serif">Please respond to<br>
Discussions about open source software for Operations Research &nbsp; &nbsp;
&nbsp; &nbsp;</font></div></table>
<br>
<td width=59%>
<table width=100%>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">To</font></div>
<td valign=top><font size=1 face="sans-serif">&lt;coin-discuss@list.coin-or.org&gt;</font>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">cc</font></div>
<td valign=top>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">Subject</font></div>
<td valign=top><font size=1 face="sans-serif">[Coin-discuss] CoinPackedMatrix:
problems appending matrices</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><font size=2><tt>I have had problems (bottom-)appending column-ordered
matrices to row-ordered matrices (using CoinPackedMatrix::bottomAppendPackedMatrix)
and (right-)appending row-ordered matrices to column-ordered matrices (using
CoinPackedMatrix::rightAppendPackedMatrix). In both cases, CoinPackedMatrix::majorAppendOrthoOrdered
is called, yielding matrices which don't look correct in terms of major
dimensions. Adding the two lines<br>
<br>
 &nbsp; majorDim_ += matrix.minorDim_;<br>
 &nbsp; size_ += matrix.size_;<br>
<br>
to the end of majorAppendOrthoOrdered() corrects this problem in my implementation.<br>
<br>
Additionally, there seems to be an indexing problem in a for loop in the
same function (line 1330 of CoinPackedMatrix.cpp) which led to a crash
in my application. The code segment reads as<br>
<br>
 &nbsp; &nbsp; for (i = 0; i &lt; matrix.minorDim_ - 1; ++i) {<br>
 &nbsp; &nbsp; &nbsp; start_[i+1] = start_[i] + CoinLengthWithExtra(orthoLength[i],
extra_gap);<br>
 &nbsp; &nbsp; }<br>
<br>
meaning that the start_ array is only adapted up to a value of i+1==matrix.minorDim-1.
However, the implementation of CoinPackedMatrix obviously assumes that
start_[matrix.minorDim_] is filled with the size of the index vector. If
I change the conditional part of the for loop to &quot;i &lt; matrix.minorDim_&quot;,
everything works fine.<br>
<br>
Could you check if I'm right? Or did I miss something?<br>
<br>
Thanks in advance,<br>
Jörg<br>
<br>
<br>
<br>
<br>
Dr. Jörg Herbers<br>
Geschäftsbereich Systeme Airport<br>
INFORM GmbH, Pascalstraße 23, 52076 Aachen, Germany<br>
Telefon: (+49) 2408 9456-24, Fax: (+49) 2408 9456-25<br>
E-Mail: joerg.herbers@inform-ac.com, Web: http://www.inform-ac.com <br>
<br>
<br>
<br>
_______________________________________________<br>
Coin-discuss mailing list<br>
Coin-discuss@list.coin-or.org<br>
http://list.coin-or.org/mailman/listinfo/coin-discuss<br>
</tt></font>
<br>