LQ0001 analyzer warning is confusing

cathei / LinqGen

Alloc-free and fast replacement for Linq, with code generation

MIT License

287 stars 12 forks source link

LQ0001 analyzer warning is confusing #17

Open NotAsea opened 9 months ago

NotAsea commented 9 months ago

This issue raise 2 problems

for further upvote of #11 ~~and add Contains support, its implementation is just Any with predicate for compare, or Where + FirstOrDefaule or even FirstOrDefault(/*predicate*/) if #11 was implemented~~
when looking in IDE problem i see I panick, have to go look it up, watch every thing in my project, only to realize after 15 minutes just your library report, maybe one remove report, two add something like LinqGen report: to clear thing more

NotAsea commented 9 months ago

after playing arround, i realize since Contains require Type must be IEquatable but IEquatable require one type must be struct or two it must be boxed, which defeat purpose of LinqGen, so after some thought , i will remove Contains support request as let user mannually handle, but i still upvote #11 and analyzer warning issue