search

EgorBo / Disasmo

VS2022 Add-in. Click on any method or class to see what .NET Core's JIT generates for them (ASM).
MIT License
638 stars 39 forks source link

Disasmo overrides project's `DefineConstants` #52

Open saucecontrol opened 6 months ago

saucecontrol commented 6 months ago

Since the dotnet build cmdline now includes /p:DefineConstants=DISASMO, this value is taken instead of any value defined in the csproj.

I like being able to detect Disasmo runs from my project and/or code, but maybe an alternate property would be better. We could then bridge that into DefineConstants ourselves if we want that.

teo-tsirpanis commented 2 months ago

Fixed by 2649e96733a29b7e9cbec2799e4ec66807971ade.