At the moment, the ugly code uses a temporary file to call LoadImage
.
We should do something better, see:
Has been done in r4637 using PIL + win32gui.SetPixelV
(setting one pixel at a time is very inefficient, but we're typically dealing with 32x32 icons here so this doesn't really matter)
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/403