Fix copy, simplify show (#114)

JuliaData · Jan 31, 2024 · 28998b1 · 28998b1
1 parent 7811b93
commit 28998b1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/DictTable.jl b/src/DictTable.jl
@@ -215,6 +215,14 @@ function Base.filter!(pred, t::DictTable)
     return t
 end
 
+function Base.copy(t::DictTable)
+    inds = copy(keys(t))
+    cols = _similar(inds, eltype(t))
+    out = DictTable{keytype(t), eltype(t), typeof(cols), typeof(inds)}(cols, inds)
+    copyto!(out, t)
+    return out
+end
+
 # Column-based operations: Some operations on rows are faster when considering columns
 
 Base.map(::typeof(identity), t::DictTable) = copy(t)

diff --git a/src/show.jl b/src/show.jl
@@ -138,7 +138,7 @@ function showtable(io::IO, @nospecialize(t), keyname)
                 else
                     col = cols[col_inds[i-1]]
                     if isassigned(col, row_ind)
-                        push!(strings[i], compact_string(t[row_ind][col_inds[i-1]]))
+                        push!(strings[i], compact_string(col[row_ind]))
                     else
                         push!(strings[i], "#undef")
                     end