Skip to content

Alias Zlib.deflate as Zlib.dump and Zlib.inflate as Zlib.load - #25

Open
casperisfine wants to merge 1 commit into
ruby:masterfrom
casperisfine:load-dump-aliases
Open

Alias Zlib.deflate as Zlib.dump and Zlib.inflate as Zlib.load#25
casperisfine wants to merge 1 commit into
ruby:masterfrom
casperisfine:load-dump-aliases

Conversation

@casperisfine

Copy link
Copy Markdown

load and dump are a very popular naming convention for serializer and such, and defining them allows easy composition.

Example:

classCoderChaindefinitialize(*coders)@coders=coders.flattenenddefdump(object)payload=object@coders.each{ |c| payload=c.dump(payload)}payloadenddefload(payload)object=payload@coders.reverse_each{ |c| object=c.load(object)}objectendendCoderChain.new(Marshal,Zlib)CoderChain.new(YAML,Snappy)

`load` and `dump` are a very popular naming convention for
serializer and such, and defining them allows easy composition.
@hsbt

hsbt commented Jan 5, 2022

Copy link
Copy Markdown
Member

I'm +1 to this change.

@akr Do you have any concerns this? I'm not sure what a background of zlib interface.

@casperisfine

Copy link
Copy Markdown
Author

?

@hsbt

hsbt commented Sep 12, 2023

Copy link
Copy Markdown
Member

It's just spam.

@hsbt

hsbt commented Sep 13, 2023

Copy link
Copy Markdown
Member

I banned Justman10000 from our organization.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@casperisfine@hsbt@byroot