From e5986aaf92aeb3afd60498dbb71075aebfa87beb Mon Sep 17 00:00:00 2001
From: otmane <otmane.mabrouk.etu@univ-lille.fr>
Date: Fri, 5 Apr 2024 23:40:39 +0200
Subject: [PATCH] I optimized the code by deleting some lines that were not
needed (creating a variable that stocks the result and then returning the
variable itself)
---
.../org/red5/server/PersistableAttributeStore.java | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/server/src/main/java/org/red5/server/PersistableAttributeStore.java b/server/src/main/java/org/red5/server/PersistableAttributeStore.java
index c00a8f50..9608dd56 100644
--- a/server/src/main/java/org/red5/server/PersistableAttributeStore.java
+++ b/server/src/main/java/org/red5/server/PersistableAttributeStore.java
@@ -248,17 +248,15 @@ public class PersistableAttributeStore extends AttributeStore implements IPersis
/** {@inheritDoc} */
@Override
public boolean setAttributes(Map<String, Object> values) {
- boolean success = super.setAttributes(values);
modified();
- return success;
+ return super.setAttributes(values);
}
/** {@inheritDoc} */
@Override
public boolean setAttributes(IAttributeStore values) {
- boolean success = super.setAttributes(values);
modified();
- return success;
+ return super.setAttributes(values);
}
/**
@@ -270,11 +268,10 @@ public class PersistableAttributeStore extends AttributeStore implements IPersis
*/
@Override
public boolean removeAttribute(String name) {
- boolean result = super.removeAttribute(name);
- if (result && name != null && !name.startsWith(IPersistable.TRANSIENT_PREFIX)) {
+ if ((super.removeAttribute(name)) && name != null && !name.startsWith(IPersistable.TRANSIENT_PREFIX)) {
modified();
}
- return result;
+ return (super.removeAttribute(name));
}
/**
--
GitLab