Page 1 of 1

Bug in latest GIT version - new parser problem?

PostPosted: 08 Mar 2011, 12:13
by vsoljan
Today I compiled (Dev-Cpp) latest version from GIT (08.03.2011.) and it compiled ok, but when I logged to FICS (tried logging as guest too) and typed "obs /b" command Winboard crashed. It happens every time I try observing a FICS game.
After that I tried version before new parser (http://git.savannah.gnu.org/cgit/xboard ... 72c13dfae7) and it works normally when trying to observe a FICS game, so I guess new parser broke something?

Re: Bug in latest GIT version - new parser problem?

PostPosted: 08 Mar 2011, 20:40
by H.G.Muller
OK, thanks. I will look into it. There still must be some things the new parser chokes on.

Re: Bug in latest GIT version - new parser problem?

PostPosted: 10 Mar 2011, 20:24
by H.G.Muller
I found the problem. I had simply forgotten to make the entry point to redirect parsing to a string (yynewstr), used only when parsing the move list fetched from an ICS. It was still an empty routine. So after getting the move list, WinBoard started to parse from a file, while no file was open. I fixed it now in the hgm.nubati git (parser branch). It should be pushed to Savannah soon.

Re: Bug in latest GIT version - new parser problem?

PostPosted: 15 Mar 2011, 10:10
by H.G.Muller
There was a additional problem parsing castling moves form the ICS. The ICS uses the non-compliant notation o-o-o here (lower-case oh), and the new parser preferred interpreting this as a Pawn push in the o-file (similar to non-compliant notations like fxg or fg, which in Xiangq, Shogi or Berolina Chess could also be fxf or ff). That should be fixed too now in Savannah master.