Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit a067b50

Browse files
committed
Get rid of minus zero in box regression test.
Commit acdf2a8 added a test case involving minus zero as a box endpoint. This is not very portable, as evidenced by the several older buildfarm members that are failing on the test because they print minus zero as just "0". If there were any significant reason to test this behavior, we could consider carrying a separate expected-file; but it doesn't look to me like there's adequate justification to accept such a maintenance burden. Just change the test to use plain zero, instead.
1 parent 2306696 commit a067b50

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/test/regress/expected/box.out

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ INSERT INTO box_temp
225225
CREATE INDEX box_spgist ON box_temp USING spgist (f1);
226226
INSERT INTO box_temp
227227
VALUES (NULL),
228-
('(-0,0)(0,100)'),
228+
('(0,0)(0,100)'),
229229
('(-3,4.3333333333)(40,1)'),
230230
('(0,100)(0,infinity)'),
231231
('(-infinity,0)(0,infinity)'),
@@ -238,7 +238,7 @@ SELECT * FROM box_temp WHERE f1 << '(10,20),(30,40)';
238238
(4,4),(2,2)
239239
(6,6),(3,3)
240240
(8,8),(4,4)
241-
(-0,100),(0,0)
241+
(0,100),(0,0)
242242
(0,Infinity),(0,100)
243243
(0,Infinity),(-Infinity,0)
244244
(7 rows)
@@ -258,7 +258,7 @@ SELECT * FROM box_temp WHERE f1 &< '(10,4.333334),(5,100)';
258258
(6,6),(3,3)
259259
(8,8),(4,4)
260260
(10,10),(5,5)
261-
(-0,100),(0,0)
261+
(0,100),(0,0)
262262
(0,Infinity),(0,100)
263263
(0,Infinity),(-Infinity,0)
264264
(8 rows)

src/test/regress/sql/box.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ CREATE INDEX box_spgist ON box_temp USING spgist (f1);
132132

133133
INSERT INTO box_temp
134134
VALUES (NULL),
135-
('(-0,0)(0,100)'),
135+
('(0,0)(0,100)'),
136136
('(-3,4.3333333333)(40,1)'),
137137
('(0,100)(0,infinity)'),
138138
('(-infinity,0)(0,infinity)'),

0 commit comments

Comments
 (0)