forked from dnSpy/dnSpy
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prettify decompiled code #311
Labels
enhancement
New feature or request
Comments
for example here is checking if secret is null so would be nice if the variable could be named something like the from public async Task<bool> Remove(string secret)
{
bool flag = secret == null;
if (flag)
{
throw new ArgumentNullException("secret");
}
return await JSPasteClient.Remove(this._key, secret);
} to public async Task<bool> Remove(string secret)
{
bool secretNull = secret == null;
if (secretNull)
{
throw new ArgumentNullException("secret");
}
return await JSPasteClient.Remove(this._key, secret);
} |
also put some new lines to some code like for ifs |
and the nullables replace it with the '?' like in normal code from this [Nullable(2)]
public string Password
{
[NullableContext(2)]
get;
[NullableContext(2)]
set;
} to this public string? Password { get; set; } |
Nullable support is available in new-ilspy branch, and local name generation might be also changed a bit |
If I get a new idea I will post it here |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem Description
Prettify code from this
to something more like this
Proposal
It would be great to prettify some ifs if the bool definition isn't too big
also would be great if the variables could get more family-friendly names like
from this
to this
Alternatives
No response
Additional Context
No response
The text was updated successfully, but these errors were encountered: