Hi, I noticed that there seems to be an inconsistency in the guacamole-common-js implementation of the protocol when compared to the manual and I don't know if it's an issue or just me misunderstanding something.
The lstroke instruction, which is documented here in the manual as taking 5 parameters ( https://guacamole.apache.org/doc/gug/protocol-reference.html#lstroke-instruction), and is implemented here with 5 parameters ( https://github.com/apache/guacamole-client/blob/062edda07b7759d3a86a9a8b476d03eb8a4aeda1/guacamole-common-js/src/main/webapp/modules/Display.js#L936-L956) seems to be called with only 2 parameters (see here https://github.com/apache/guacamole-client/blob/062edda07b7759d3a86a9a8b476d03eb8a4aeda1/guacamole-common-js/src/main/webapp/modules/Client.js#L1234-L1243 ). The interesting part is that while display.strokeLayer is only called with 2 parameters, those parameters also are potentially out of order since the first argument to strokeLayer appears correctly to be the layer to draw upon, the second argument appears like it should be the fifth argument, the srcLayer. I'm just wondering if this is correct or if I'm just missing something. Sean
