Changed class Optional to struct to reduce heap allocations.

yreynhout / AggregateSource

Lightweight infrastructure for doing eventsourcing using aggregates

BSD 3-Clause "New" or "Revised" License

257 stars 60 forks source link

Changed class Optional to struct to reduce heap allocations. #5

Closed martijnvdbrk closed 11 years ago

yreynhout commented 11 years ago

Might be a good idea to resurface the Equals/GetHashCode method because T could be a reference type and it seems ValueType is using reflection to compare fields (http://msdn.microsoft.com/en-us/library/2dts52z7.aspx). But that's just me being picky ;-)

martijnvdbrk commented 11 years ago

I agree, and removing Equals was actually not my intent. Too many CQRS beers i guess...