Maybe a little bug in Xboard 4.9.1

[hgm]

I pushed an attempt to cure this to my source repository at winboard.nl. This breaks a line at the first space after the input buffer is half full.

[chrjly2]

Hello together, hello @hgm, (How can I adress hgm in a forum post?)

I got a problem wit Xboard 4.9.1 (on Macosx) that I cannot load pgns, that have to many characters in one line.

E.g.: https://ccrl.live/pgns/The_Great_Kiwi_H ... el_2.0.pgn

I have to download and split them manually into a couple of lines.
If this a general issue, and if yes, maybe there could be a fix for it?

It would be very very kind!
Thank you.

I have the same problem with my xboard 4.9.1 when I load a pgn file downloaded from the CCRL live viewer.
Here is a sed command that format the pgn file into a one move per line pgn file:

sed -E 's/[0-9]+\. /\n&/g' input_file.pgn >output_file.pgn

Christophe

[Roland Chastain]

I pushed an attempt to cure this to my source repository at winboard.nl. This breaks a line at the first space after the input buffer is half full.

Thank you. There would be a small correction to do:

Code: Select all

parser.c: In function 'ReadLine':
parser.c:292:43: error: 'PARSEBUF' undeclared (first use in this function); did you mean 'PARSEBUFSIZE'?
  292 |         if(c == ' ' && inPtr - inputBuf > PARSEBUF/2) break;  // break too-long lines
      |                                           ^~~~~~~~
      |                                           PARSEBUFSIZE

[hgm]

Sorry. I thought I had tested the patch by running "make", and that it compiled without errors. I must have done that before I actually saved the modified file from the editor.

Anyway, it should be fixed now.

[Dave Gomboc]

Sorry. I thought I had tested the patch by running "make", and that it compiled without errors. I must have done that before I actually saved the modified file from the editor.

Anyway, it should be fixed now.

@hgm Hey, H.G., when you have a chance to reply to my personal email (from ~10 days ago), please also indicate whether these newest changes being discussed here should be or not be included in whatever level of permission you choose to provide. The sooner the issue in the email discussion gets resolved, the sooner I will be able to actually create a new point release for XBoard.