CSharpier removes blank line between if statement and unsafe block

[fgimian]

Hey there, hope you're doing great! 😄 Thank you so much for this amazing tool, absolutely love it!

I only encountered one minor issue while formatting my codebase:

    public void Deactivate()
    {
        if (!_attached)
        {
            return;
        }
        unsafe
        {
            uint currentThreadId = PInvoke.GetCurrentThreadId();
            uint targetThreadId = PInvoke.GetWindowThreadProcessId(Handle, lpdwProcessId: null);
            if (PInvoke.AttachThreadInput(currentThreadId, targetThreadId, fAttach: false) != 0)
            {
                _attached = false;
            }
        }
    }

Notice that CSharpier removed the blank line between my if and unsafe blocks. It doesn't do this for other block types (e.g. if I have multiple if statements after one another, so I suspect that maybe the unsafe keyword was not accounted for?

Thank you so much in advance!
Fotis

[belav]

Thank you so much for this amazing tool, absolutely love it!

You're welcome, I'm happy to hear that!

so I suspect that maybe the unsafe keyword was not accounted for?

This was pretty much it. Just a single line fix, unsafe slipped through when having CSharpier take into account existing new lines between statements.

Thanks for reporting it! The 0.25 release should be out soonish.

[fgimian]

Thank you so much for this amazing tool, absolutely love it!

You're welcome, I'm happy to hear that!

so I suspect that maybe the unsafe keyword was not accounted for?

This was pretty much it. Just a single line fix, unsafe slipped through when having CSharpier take into account existing new lines between statements.

Thanks for reporting it! The 0.25 release should be out soonish.

Thank you so much for your help! 😄

Cheers
Fotis