DON'T MERGE - test CI failure

microsoft · Jan 17, 2025 · 263aced · 263aced
1 parent 3cb8919
commit 263aced
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/include/ccf/http_accept.h b/include/ccf/http_accept.h
@@ -45,21 +45,28 @@ namespace ccf::http
 
     bool operator<(const AcceptHeaderField& other) const
     {
-      static constexpr auto float_comp_epsilon = 0.0000001f;
-      if (abs(q_factor - other.q_factor) > float_comp_epsilon)
+      if (q_factor != other.q_factor)
       {
         return q_factor < other.q_factor;
       }
 
-      if (is_wildcard(mime_type) && !is_wildcard(other.mime_type))
+      if (is_wildcard(mime_type))
       {
         return true;
       }
+      else if (is_wildcard(other.mime_type))
+      {
+        return false;
+      }
 
-      if (is_wildcard(mime_subtype) && !is_wildcard(other.mime_subtype))
+      if (is_wildcard(mime_subtype))
       {
         return true;
       }
+      else if (is_wildcard(other.mime_subtype))
+      {
+        return false;
+      }
 
       // Spec says these mime types are now equivalent. For stability, we
       // order them lexicographically