assert: simplify isNil implementation

stretchr / testify

A toolkit with common assertions and mocks that plays nicely with the standard library

MIT License

23.15k stars 1.59k forks source link

Closed dolmen closed 8 months ago

dolmen commented 10 months ago

Simpler and faster (also less allocations) in internal function isNil.

Simplify isNil: use a switch on reflect.Value.Kind() instead of browsing a slice of allowed Kinds.

Less code, less complexity, less memory usage, more speed.

dolmen commented 8 months ago

@MovieStoreGuy Could you review this?