From 4f3330d40819630b3de9f1a68aa8af8255e3c480 Mon Sep 17 00:00:00 2001
From: Max Horn <max@quendi.de>
Date: Mon, 3 Feb 2025 14:10:58 +0100
Subject: [PATCH] FakeFmpqMat: don't store rows & cols

They are already stored in the wrapped ZZMatrix
---
 src/HeckeTypes.jl                | 10 ----------
 src/LinearAlgebra/FakeFmpqMat.jl |  2 --
 2 files changed, 12 deletions(-)

diff --git a/src/HeckeTypes.jl b/src/HeckeTypes.jl
index 58688843c1..63f39bc466 100644
--- a/src/HeckeTypes.jl
+++ b/src/HeckeTypes.jl
@@ -526,8 +526,6 @@ Used predominantly to represent bases of orders in absolute number fields.
 mutable struct FakeFmpqMat
   num::ZZMatrix
   den::ZZRingElem
-  rows::Int
-  cols::Int
 
   function FakeFmpqMat()
     z = new()
@@ -538,8 +536,6 @@ mutable struct FakeFmpqMat
     z = new()
     z.num = x
     z.den = y
-    z.rows = nrows(x)
-    z.cols = ncols(x)
     if !simplified
       simplify_content!(z)
     end
@@ -550,8 +546,6 @@ mutable struct FakeFmpqMat
     z = new()
     z.num = x[1]
     z.den = x[2]
-    z.rows = nrows(x[1])
-    z.cols = ncols(x[1])
     if !simplified
       simplify_content!(z)
     end
@@ -563,15 +557,11 @@ mutable struct FakeFmpqMat
     z = new()
     z.num = x
     z.den = one(ZZ)
-    z.rows = nrows(x)
-    z.cols = ncols(x)
     return z
   end
 
   function FakeFmpqMat(x::QQMatrix)
     z = new()
-    z.rows = nrows(x)
-    z.cols = ncols(x)
 
     n, d = _fmpq_mat_to_fmpz_mat_den(x)
 
diff --git a/src/LinearAlgebra/FakeFmpqMat.jl b/src/LinearAlgebra/FakeFmpqMat.jl
index 1a38433fba..958d3bc275 100644
--- a/src/LinearAlgebra/FakeFmpqMat.jl
+++ b/src/LinearAlgebra/FakeFmpqMat.jl
@@ -357,8 +357,6 @@ function Base.deepcopy_internal(x::FakeFmpqMat, dict::IdDict)
   z = FakeFmpqMat()
   z.num = Base.deepcopy_internal(x.num, dict)
   z.den = Base.deepcopy_internal(x.den, dict)
-  z.rows = nrows(x)
-  z.cols = ncols(x)
   return z
 end