Re: [Bug-XBoard] premove bug?

[Eric Mullins]

Eric Mullins wrote:
> On FICS, $<foo> prevents alias substitution.  backend.c attempts to 
> set the prefix (for preventing alias substitution) based on the ICS 
> type.  I don't know how well that works outside of FICS since that's 
> all I ever use.  I'm not sure which part of the code contains the 
> prefix for premove, but it should be using the ics_prefix variable 
> instead of '$' in order to presumably use the correct one based on ICS 
> type.

As an aside: from a design perspective, this behavior should be 
encapsulated in the SendToICS routine().  neither frontend nor backend 
code should ever have to concern itself with prepending a prefix when 
issuing server commands.  Yet pretty much every command we issue to the 
server technically should be prepended with a prefix if availabe.  (You 
don't want 'set width 40' to become something else when the user has an 
alias for 'set')