Thanks for the bug fixes! Depressingly I haven't compiled Quackle in a few months. I would like to try your new stuff and help you submit the bug fixes but I'm going on vacation for two weeks!
If you let me know your Sourceforge account details though I can let you submit your changes yourself :) Then hopefully we can make a new Quackle release sometime! Thanks, Jason On Sat, Jun 5, 2010 at 11:50, Tom Sirgedas <[email protected]> wrote: > > > In Reporter::reportPosition(), > > boardLineBeginning = boardIt + 1; > reportLineBeginning = reportIt + 1; > > crashes my debug build, since the "+ 1" is meant to skip the '\n' > character, but the report doesn't have a '\n' at the very end. > > This change seems to fix it for me: > > // skip '\n' > if ( boardIt != boardEnd ) > boardLineBeginning = boardIt + 1; > > // skip '\n' > if ( reportIt != reportEnd ) > reportLineBeginning = reportIt + 1; > > Also... When I ask "Championship Player" for a full report, I get a crash. > I changed this code: > > void Simulator::moveConsideredMovesToBeginning(MoveList &moves) const > { > for (MoveList::const_iterator consideredIt = m_consideredMoves.begin(); > consideredIt != m_consideredMoves.end(); ++consideredIt) > { > for (int i = 0; i < (int) moves.size(); ++i) > { > if (*consideredIt == moves[i]) > { > moves.erase(moves.begin()+i); > moves.insert(moves.begin(), *consideredIt); > } > } > } > } > > FWIW, I'm using Windows XP > > Tom Sirgedas > > >
