Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-23 01:21:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

[wpiutil] DataLog.addSchema(): Don't add into a set view (#5829)

Browse Source
This commit is contained in:
Peter Johnson
2023-10-26 19:17:47 -07:00
committed by GitHub
parent 2ab4fcbc24
commit b91317fd36
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
wpiutil/src/main/java/edu/wpi/first/util/datalog/DataLog.java
View File

@@ -118,7 +118,7 @@ public final class DataLog implements AutoCloseable {
* @return True if schema already registered
*/
public boolean hasSchema(String name) {
return m_schemaSet.contains(name);
return m_schemaMap.containsKey(name);
}
/**
@@ -134,7 +134,7 @@ public final class DataLog implements AutoCloseable {
* @param timestamp Time stamp (may be 0 to indicate now)
*/
public void addSchema(String name, String type, byte[] schema, long timestamp) {
if (!m_schemaSet.add(name)) {
if (m_schemaMap.putIfAbsent(name, 1) != null) {
return;
}
DataLogJNI.addSchema(m_impl, name, type, schema, timestamp);
@@ -168,7 +168,7 @@ public final class DataLog implements AutoCloseable {
* @param timestamp Time stamp (may be 0 to indicate now)
*/
public void addSchema(String name, String type, String schema, long timestamp) {
if (!m_schemaSet.add(name)) {
if (m_schemaMap.putIfAbsent(name, 1) != null) {
return;
}
DataLogJNI.addSchemaString(m_impl, name, type, schema, timestamp);
@@ -506,5 +506,4 @@ public final class DataLog implements AutoCloseable {
private long m_impl;
private final ConcurrentMap<String, Integer> m_schemaMap = new ConcurrentHashMap<>();
private final Set<String> m_schemaSet = m_schemaMap.keySet();
}