Jump to content
e.c

OverbyteIcsWebSocketCli: Adjustment suggestion

Recommended Posts

Good morning !

 

We had a little trouble debugging this way. Here is a suggested adjustment, if possible.

 

At OverbyteIcsWebSocketCli.pas, line 954:
            ... 
            if State in [wsfsCompleted, wsfsNotComplete] then begin
                j := 0;
                for i := 0 to DataBytesInBuf-1 do begin
                  { PData^ := PData^ xor Mask[j]; // Original
                  } {$R-} PData^ := PData^ xor Mask[j]; {$R+} // Suggestion
                    Inc(j);
                    if j > 3 then
                        j := 0;
                end;
            end;
            ...

Thank you for your attention.

Share this post


Link to post

Thanks, I'll look at this when I get back to ICS next week. 

 

Note, there is an ICS forum for future support, but you don't need to repost this topic.

 

Angus

 

  • Like 2

Share this post


Link to post
if State in [wsfsCompleted, wsfsNotComplete] then begin
    j := 0;
    for i := 0 to DataBytesInBuf - 1 do begin
        {$R-}
        PData^ := PData^ xor Mask[j];
        {$R+}
        Inc(j);
        if j > 3 then
            j := 0;
    end;
end;

Try this.

Share this post


Link to post
Quote

Try this. 

Your code looks the same as the original suggestion, which was added to the version in SVN last month.  There are other fixes to this unit pending. 

 

Seems to be quite an interest in the new ICS websocket components. 

 

Angus

 

Share this post


Link to post

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×