[openrtm-commit:02738] r1002 - in trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC: executionContext util
openrtm @ openrtm.org
openrtm @ openrtm.org
2017年 8月 10日 (木) 17:08:45 JST
Author: t-katami
Date: 2017-08-10 17:08:45 +0900 (Thu, 10 Aug 2017)
New Revision: 1002
Modified:
trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/executionContext/PeriodicECOrganization.java
trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/util/Properties.java
Log:
[compat,bugfix,->RELENG_1_2] Fixed the items pointed out by the static code analysis tool.
Modified: trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/executionContext/PeriodicECOrganization.java
===================================================================
--- trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/executionContext/PeriodicECOrganization.java 2017-08-10 04:53:40 UTC (rev 1001)
+++ trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/executionContext/PeriodicECOrganization.java 2017-08-10 08:08:45 UTC (rev 1002)
@@ -598,10 +598,9 @@
String plist
= m_rtobj.getProperties().getProperty("conf.default.exported_ports");
+ plist = plist.replace(" ,",",");
+ plist = plist.replace(", ",",");
m_expPorts = StringUtil.split(plist, ",");
- for (int i=0; i<m_expPorts.size(); ++i) {
- m_expPorts.set(i,m_expPorts.get(i).trim());
- }
}
/**
Modified: trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/util/Properties.java
===================================================================
--- trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/util/Properties.java 2017-08-10 04:53:40 UTC (rev 1001)
+++ trunk/OpenRTM-aist-Java/jp.go.aist.rtm.RTC/src/jp/go/aist/rtm/RTC/util/Properties.java 2017-08-10 08:08:45 UTC (rev 1002)
@@ -721,11 +721,10 @@
public void clear() {
while (true) {
- int ic = this.leaf.size() - 1;
- if(ic < 0) {
+ if(this.leaf.size() <= 0) {
break;
}
- Properties tail = this.leaf.get(ic);
+ Properties tail = this.leaf.lastElement();
if (tail != null) {
tail.destruct();
}
More information about the openrtm-commit
mailing list