chmln / sd

Intuitive find & replace CLI (sed alternative)

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Need to document that `sd` does _not_ stream, loads entire input into memory

corneliusroemer opened this issue · comments

I was happy with sd until I noticed that it reads the entire input into memory. There's no streaming support at all.

This is mentioned in the following issues:

But I would expect this to be documented in the README, as it is a major caveat and limitation compared to standard sed.

Don't we mmap everything now? @nc7s

@dev-ardi:

Don't we mmap everything now? @nc7s

Stdin is still read in full into memory, it's then converted (wrapped?) to a mmap. See discussion in #287.