Blob Blame History Raw
From 2aa02353663674a338c92fa4e72a0c0e529c410e Mon Sep 17 00:00:00 2001
From: Enrico Guiraud <enrico.guiraud@cern.ch>
Date: Fri, 8 Apr 2022 17:52:18 +0200
Subject: [PATCH] [DF] Fix long int + Snapshot test on 32 bit platforms

Co-authored-by: Mattias Ellert <mattias.ellert@physics.uu.se>
---
 tree/dataframe/test/dataframe_snapshot.cxx | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tree/dataframe/test/dataframe_snapshot.cxx b/tree/dataframe/test/dataframe_snapshot.cxx
index f41848e96a..ddbacaba0f 100644
--- a/tree/dataframe/test/dataframe_snapshot.cxx
+++ b/tree/dataframe/test/dataframe_snapshot.cxx
@@ -495,11 +495,14 @@ void ReadWriteCarray(const char *outFileNameBase)
    t.Branch("vb", vb, "vb[size]/O");
    t.Branch("vl", vl, "vl[size]/G");
 
+   // use 2**33 as a larger-than-int value on 64 bits, otherwise just something larger than short (2**30)
+   static constexpr long int longintTestValue = sizeof(long int) == 8 ? 8589934592 : 1073741824;
+
    // Size 1
    size = 1;
    v[0] = 12;
    vb[0] = true;
-   vl[0] = 8589934592; // 2**33
+   vl[0] = longintTestValue;
    t.Fill();
 
    // Size 0 (see ROOT-9860)
@@ -546,7 +549,7 @@ void ReadWriteCarray(const char *outFileNameBase)
       EXPECT_EQ(rvb.GetSize(), 1u);
       EXPECT_TRUE(rvb[0]);
       EXPECT_EQ(rvl.GetSize(), 1u);
-      EXPECT_EQ(rvl[0], 8589934592);
+      EXPECT_EQ(rvl[0], longintTestValue);
 
       // Size 0
       EXPECT_TRUE(r.Next());
-- 
2.35.1