jgm/pandoc-types

Why do we need isNull?

Closed this issue · 1 comments

josephcsible commented

pandoc-types/Text/Pandoc/Builder.hs

Lines 196 to 197 in 154b91b

isNull :: Many a -> Bool
isNull = Seq.null . unMany

Is there a reason that we need isNull? Many is Foldable, so isn't it always the same as null? Can we mark it as deprecated now and eventually remove it?

jgm commented

I think you're right. Yes, we could deprecate it. We should also replace places that use it with null.