AnimatedGif icon indicating copy to clipboard operation
AnimatedGif copied to clipboard

An in memory option would be nice

Open BasHamer opened this issue 6 years ago • 3 comments

the part of my code that picks the file names is a bit separated from where I create the files, it would be nice if I could use the library in memory w/o file I/O

BasHamer avatar Oct 02 '18 20:10 BasHamer

You could create a pull request for that if you want. Otherwise, I could look into that sometimes next week or so.

mrousavy avatar Oct 03 '18 04:10 mrousavy

If a clean way comes to mind I'll take a look at it; as it stands the constructor has the file IO build into it for the AnimatedGifCreator and the IDisposable triggers the file write. It seems clunky to put it in there, as this really could be a factory method that is configured w/ defaults and then sits there waiting for files; leaving the original creator as a wrapper on top of it that handles file IO.

Not a small change, plus a breaking change if you want to name things what they are.

BasHamer avatar Oct 03 '18 16:10 BasHamer

I could redesign the library if there is enough interest, then an in memory option wouldn't be that much more work, as I'll be using the Stream abstract base class. See the redesign project for more info. Help is greatly appreciated 😉👍

mrousavy avatar Oct 08 '18 08:10 mrousavy