search

jackdreilly / deferrable

Defer widget cleanup operations automagically to the `dispose()` method
BSD 2-Clause "Simplified" License
1 stars 0 forks source link

naming rec: defer -> deferCleanup #1

Open asmengistu opened 4 years ago

asmengistu commented 4 years ago

On first read I thought defer(fn()) executes fn in some deferred manner (as opposed to deferring the clean up). Thus I suggest deferCleanup as an improvement, there may be an even better name, but this is what I got.

jackdreilly commented 4 years ago

the deferrable package maintainer happily reviews PR's ;)

just kidding, that's a good suggestion, maybe onDispose(fn) ?

On Mon, May 4, 2020 at 6:53 PM Abel Mengistu notifications@github.com wrote:

On first read I thought defer(fn()) executes fn in some deferred manner (as opposed to deferring the clean up). Thus I suggest deferCleanup as an improvement, there may be an even better name, but this is what I got.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jackdreilly/deferrable/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEXK6ZK6QTT6EQGO32SW5TRP3XJNANCNFSM4MY5IR2Q .

asmengistu commented 4 years ago

autoDisposed(fn)?