sikuli-driver team mailing list archive
-
sikuli-driver team
-
Mailing list archive
-
Message #49887
[Bug 1809881] Re: [1.1.4] VNCScreen::stop() should remove the related resources --- fixed 2018-12-28 build#158
** Description changed:
+ ----------------- the below code is from version 1.1.3 !!!
+ ----------------- the fixes are in 1.1.4 https://github.com/RaiMan/SikuliX1
+
+ ---------------------------------------------------------
+
This piece of code below keeps the VNCScreen object in the HashMap.
https://github.com/RaiMan/SikuliX-2014/blob/develop/API/src/main/java/org/sikuli/vnc/VNCScreen.java
from lines 59:
public void stop() {
try {
close();
} catch (IOException e) {
Debug.error("VNCScreen: stop: %s", e.getMessage());
}
screens.put(this, null);
}
Need a public api so that the VNCScrren Object can be removed from the cache when stop() is called.
Cannot use stopAll as the scenario where this is being used there can be mutiple VNCScreens and need to remove the specific VNCScreen object with disturbing or removing other VNCScrren objects.
Sikuli Version being used is 1.1.4. Operating system being used is
Windows 64 bit.
Multiple creation of VNCScreen Object with same VNCClient. The hash map overflows ie the Java responds with out of memory exception.
To reproduce keep the max size of heap of java to be 1GB , create mutiple vncscreen objects with same client and run that in a loop of 1000000.
** Description changed:
----------------- the below code is from version 1.1.3 !!!
- ----------------- the fixes are in 1.1.4 https://github.com/RaiMan/SikuliX1
+ ----------------- the fixes are in 1.1.4
+ https://github.com/RaiMan/SikuliX1
---------------------------------------------------------
This piece of code below keeps the VNCScreen object in the HashMap.
https://github.com/RaiMan/SikuliX-2014/blob/develop/API/src/main/java/org/sikuli/vnc/VNCScreen.java
from lines 59:
public void stop() {
try {
close();
} catch (IOException e) {
Debug.error("VNCScreen: stop: %s", e.getMessage());
}
screens.put(this, null);
}
Need a public api so that the VNCScrren Object can be removed from the cache when stop() is called.
Cannot use stopAll as the scenario where this is being used there can be mutiple VNCScreens and need to remove the specific VNCScreen object with disturbing or removing other VNCScrren objects.
Sikuli Version being used is 1.1.4. Operating system being used is
Windows 64 bit.
Multiple creation of VNCScreen Object with same VNCClient. The hash map overflows ie the Java responds with out of memory exception.
To reproduce keep the max size of heap of java to be 1GB , create mutiple vncscreen objects with same client and run that in a loop of 1000000.
--
You received this bug notification because you are a member of Sikuli
Drivers, which is subscribed to Sikuli.
https://bugs.launchpad.net/bugs/1809881
Title:
[1.1.4] VNCScreen::stop() should remove the related resources ---
fixed 2018-12-28 build#158
Status in Sikuli:
Fix Committed
Bug description:
----------------- the below code is from version 1.1.3 !!!
----------------- the fixes are in 1.1.4
https://github.com/RaiMan/SikuliX1
---------------------------------------------------------
This piece of code below keeps the VNCScreen object in the HashMap.
https://github.com/RaiMan/SikuliX-2014/blob/develop/API/src/main/java/org/sikuli/vnc/VNCScreen.java
from lines 59:
public void stop() {
try {
close();
} catch (IOException e) {
Debug.error("VNCScreen: stop: %s", e.getMessage());
}
screens.put(this, null);
}
Need a public api so that the VNCScrren Object can be removed from the cache when stop() is called.
Cannot use stopAll as the scenario where this is being used there can be mutiple VNCScreens and need to remove the specific VNCScreen object with disturbing or removing other VNCScrren objects.
Sikuli Version being used is 1.1.4. Operating system being used is
Windows 64 bit.
Multiple creation of VNCScreen Object with same VNCClient. The hash map overflows ie the Java responds with out of memory exception.
To reproduce keep the max size of heap of java to be 1GB , create mutiple vncscreen objects with same client and run that in a loop of 1000000.
To manage notifications about this bug go to:
https://bugs.launchpad.net/sikuli/+bug/1809881/+subscriptions
References