WurstStdlib2 icon indicating copy to clipboard operation
WurstStdlib2 copied to clipboard

LinkedListIndexed

Open WareditorGit opened this issue 6 years ago • 12 comments

A module that turns a class into a collection of linked lists where each list is composed of instances of the same index.

This will be useful for closureFrames and closureKeyPresses.

Not sure about the name.

WareditorGit avatar Jul 12 '19 10:07 WareditorGit

This will be useful for closureFrames and closureKeyPresses.

It should also be able to be used in ClosureEvents right?

Frotty avatar Jul 12 '19 15:07 Frotty

Yeah and others if needed

WareditorGit avatar Jul 12 '19 15:07 WareditorGit

@WareditorGit can you fix eof lf? should adhere to editorconfig. Did you have any further ideas about other names?

Frotty avatar Jul 21 '19 08:07 Frotty

Sorry, I am away from my comp during this period. Will work on this when I get back.

WareditorGit avatar Jul 25 '19 22:07 WareditorGit

@WareditorGit I could also merge like this or are there changes u want to make besides name?

Frotty avatar Aug 01 '19 21:08 Frotty

No changes. However I agree that it's might be too peculiar to include in the std lib and might add some unrequired abstraction.

WareditorGit avatar Aug 02 '19 17:08 WareditorGit

What's the point of not having it in stdlib if we plan to use it in stdlib?

Frotty avatar Aug 07 '19 15:08 Frotty

I am saying maybe not use it if it's only for those few cases.

WareditorGit avatar Aug 07 '19 17:08 WareditorGit

Will this work for destroying listeners during a callback without temp var? I still want to merge tbh.

Frotty avatar Sep 29 '19 19:09 Frotty

bump

Frotty avatar Dec 07 '19 10:12 Frotty

What's the point of not having it in stdlib if we plan to use it in stdlib?

If you plan to use in stdlib then it should be in stdlib.

It would be interesting to see a concrete stdlib change - in this PR - that moves old code to using this new feature.

Cokemonkey11 avatar Jan 06 '20 23:01 Cokemonkey11

I actually like this, but the hashmaps seem overkill, why not just use module attribute. Might fix and merge.

Frotty avatar Apr 15 '21 20:04 Frotty