On 2014/10/23 15:09, Michael Tokarev wrote: > On 10/23/2014 09:39 AM, arei.gong...@huawei.com wrote: >> From: Gonglei <arei.gong...@huawei.com> >> >> vnc_display_close() is called in vnc_display_open(). >> And in this moment, the ds variable is initilized >> at vnc_display_init(). Calling vnc_display_close() >> is superfluous. > > Maybe the intent was to be able to open/close new vnc > displays in the future? I think I've seen patches to > allow several vnc displays at once... >
I also saw Gerd's patch series for multiple vnc server instances. But every vnc server has its own vnc status, they shouldn't influence with each other. :) This logic is the same, initialize and close immediately (the close process is superfluous). Best regards, -Gonglei
