Re: [Scidvspc-users] Bug between ScidVsPC and Stockfish

[Alex W. <al...@un...>]

On 4/22/2016 8:26 PM, Steve A wrote:
> I cant see the bug report, even after joining their bugzilla.
>
> But if they are happy to have their app crash with very reasonable
> input... i guess we should probably add a safeguard to handle it.
> Stockfish does many unfriendly things in a way *no* other engine does.
>
> This should handle it (also attached) and will probably get comitted,
> but as my version of SF doesnt crash, i cant test it.
>
> Index: tcl/tools/analysis.tcl
> ===================================================================
> --- tcl/tools/analysis.tcl    (revision 2523)
> +++ tcl/tools/analysis.tcl    (working copy)
> @@ -3544,6 +3544,9 @@
>           append cmd { [ } " after $delay sendPosToEngineUCI $n $delay " { ] }
>           set analysis(after$n) [eval [list after idle $cmd]]
>       } else {
> +        if {[sc_pos moves] == {}} {
> +          return
> +        }
>           ### Dont send position if annotating and in book
>       if { $::annotate(Engine) == $n && ! $::wentOutOfBook &&
> $::useAnalysisBook} {
>         bookAnnotation

Sorry about that.  I just updated the bug 
https://bugzilla.redhat.com/show_bug.cgi?id=1325013 so that it is public.

Thanks for the patch.  On Monday, I can drop that patch into the RPM 
that I've built for Fedora, rebuild it, and test it out.  Thanks for the 
quick response!
------------
Regards,
Alex

P.S. I'm 99.99% sure I'm subscribed properly to the mailing list, but 
Steve, if you have to moderate this email let me know and I'll try to 
get it figured out.