From f8a06d78f31837f316b117461b675400b530b90c Mon Sep 17 00:00:00 2001
From: Dhanya Thattil <dhanya.thattil@psi.ch>
Date: Tue, 1 Jul 2025 17:00:59 +0200
Subject: [PATCH] refactor cmd parsing (detid can be parsed directly)

---
 slsDetectorSoftware/src/CallerSpecial.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/slsDetectorSoftware/src/CallerSpecial.cpp b/slsDetectorSoftware/src/CallerSpecial.cpp
index 174c51276..10330f1b6 100644
--- a/slsDetectorSoftware/src/CallerSpecial.cpp
+++ b/slsDetectorSoftware/src/CallerSpecial.cpp
@@ -753,13 +753,8 @@ std::string Caller::rx_roi(int action) {
         if (!args.empty()) {
             WrongNumberOfParameters(0);
         }
-        if (det_id != -1) {
-            auto t = det->getRxROI(det_id);
-            os << ToString(t) << '\n';
-        } else {
-            auto t = det->getRxROI();
-            os << ToString(t) << '\n';
-        }
+        auto t = det->getRxROI(det_id);
+        os << ToString(t) << '\n';
     } else if (action == defs::PUT_ACTION) {
         std::vector<defs::ROI> rois;
         // Support multiple args with bracketed ROIs, or single arg with