search

sestoft / C5

C5 generic collection library for C#/.NET
http://www.itu.dk/research/c5/
MIT License
1.04k stars 179 forks source link

default constructors #68

Closed FruitcakeElemental closed 4 years ago

FruitcakeElemental commented 6 years ago

Updated collections to have a default constructor where missing, to make them compatible with new() generic constraints.

ondfisk commented 5 years ago

Thanks for the PR. We have removed the MemoryType stuff from C5 altogether so default constructors are back. It's in NuGet as a 3.0.0-beta release. Hope that helps (and sorry for taking a full year to respond)

FruitcakeElemental commented 5 years ago

Ok cool, but that memory type stuff was one of the main reasons I'm using C5. I looked at the pull request, what changes in unity are you referring to?

ondfisk commented 5 years ago

This is a week old quote from Giovanni who implemented the memory stuff in the first place: "Since Unity 5.6 they updated to the latest version of the Mono Runtime. Thus Unity doesn't suffer from that garbage generation issue anymore. Feel free to remove all of the memory safe stuff."