StefH / MediaInfo.DotNetWrapper

A C# .NET Wrapper for the MediaInfo.Native and MediaInfo.dll
MIT License
37 stars 8 forks source link

Use source generation for dll imports #10

Open SbiCA opened 1 year ago

SbiCA commented 1 year ago

Hi @StefH ,

First of all thanks for you work and this library 🎉

Have you thought on using the source generation introduced in .NET 7 and maybe adding some benchmarks?

https://learn.microsoft.com/en-us/dotnet/standard/native-interop/cross-platform https://learn.microsoft.com/en-us/dotnet/standard/native-interop/pinvoke-source-generation

Cheers Simon

StefH commented 1 year ago

@SbiCA This looks interesting...

If you have time, you can also start on a PR for this?

StefH commented 1 year ago

@SbiCA Do you have time to start a PR or provide some more example?

SbiCA commented 1 year ago

@StefH sorry for the delay I'm afraid not yet and I don't think I have time this year to help you on that. There are a few good talks on source generation e.g. https://www.youtube.com/watch?v=azJm_Y2nbAI . I m also happy to jump on a slack/hangouts call and discuss possible approaches if you have some time available but from my end it will be hard to contribute code the next 3 months