From 54cffac521c1cbc58a0302002dd5793742bcfac0 Mon Sep 17 00:00:00 2001
From: csoler <csoler@users.sourceforge.net>
Date: Sun, 8 Jan 2023 10:44:32 +0100
Subject: [PATCH] simplified logic of interval calculation for exact size
 search

---
 .../src/gui/advsearch/expressionwidget.cpp      | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/retroshare-gui/src/gui/advsearch/expressionwidget.cpp b/retroshare-gui/src/gui/advsearch/expressionwidget.cpp
index dc1738825..7745c8a60 100644
--- a/retroshare-gui/src/gui/advsearch/expressionwidget.cpp
+++ b/retroshare-gui/src/gui/advsearch/expressionwidget.cpp
@@ -187,21 +187,12 @@ RsRegularExpression::Expression* ExpressionWidget::getRsExpression()
 
                 if(cond == RsRegularExpression::Equals)
                 {
-                    // Now compute a proper interval. For that we look for the largest non significant digit, and add/remove 50% of it.
-                    int i=0;
-                    while(!(s & (1<<i)) && i<32)
-                        i++;
+                    // Now compute a proper interval. There is no optimal solution, so we aim for the simplest: add/remove 20% of the initial value.
 
-                    if(i==0)
-                        if(MB)
-                            expr = new RsRegularExpression::SizeExpressionMB(RsRegularExpression::Equals, (int)s) ;
-                        else
-                            expr = new RsRegularExpression::SizeExpression(RsRegularExpression::Equals, (int)s) ;
+                    if(MB)
+                        expr = new RsRegularExpression::SizeExpressionMB(RsRegularExpression::InRange, (int)(s*0.8) , (int)(s*1.2));
                     else
-                        if(MB)
-                            expr = new RsRegularExpression::SizeExpressionMB(RsRegularExpression::InRange, (int)s - (1<<(i-1)), (int)s + (1<<(i-1)));
-                        else
-                            expr = new RsRegularExpression::SizeExpression(RsRegularExpression::InRange, (int)s - (1<<(i-1)), (int)s + (1<<(i-1)));
+                        expr = new RsRegularExpression::SizeExpression(RsRegularExpression::InRange, (int)(s*0.8) , (int)(s*1.2));
                 }
                 else
                 {