openai / go-vncdriver

Fast VNC driver
MIT License
123 stars 60 forks source link

Maybe ditch libjpeg #3

Closed jeremyschlatter closed 7 years ago

jeremyschlatter commented 7 years ago

We added libjpeg because my first implementation of tight decoding was really slow at decompressing jpeg. In particular, it spent a lot of time in image.At, and I have since found out that the Go authors recommend not using that function, and using the underlying Image implementation (image.RGBA or whatever else it happens to be in our case) instead, because the underlying implementation will be much faster.

Using the recommended Go implementation might be fast enough, and it will simplify installation of this package.

jeremyschlatter commented 7 years ago

Closing in favor of #6