Skip to content

Conversation

@lonitra
Copy link
Member

@lonitra lonitra commented Jul 19, 2022

  • Add HDC conversion to scopes
  • Remove unused BeginPath
Microsoft Reviewers: Open in CodeFlow

- Add HDC conversion to scopes
- Remove unused BeginPath
@lonitra lonitra requested a review from a team as a code owner July 19, 2022 02:12
@ghost ghost assigned lonitra Jul 19, 2022
Copy link
Member

@JeremyKuhne JeremyKuhne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Welcome back!!

@elachlan elachlan mentioned this pull request Jul 19, 2022
Copy link
Contributor

@RussKie RussKie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 with stylistics nits

GetCurrentProcessId
GlobalReAlloc
GlobalFree
BitBlt
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar to #7444 (comment) - please have this file sorted in alphabetical order.

Comment on lines 13 to 21
IHandle hdc,
int x,
int y,
int cx,
int cy,
HDC hdcSrc,
int x1,
int y1,
ROP_CODE rop)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did the indentation change?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops! Must've happened by accident when I was moving things around.

Comment on lines 10 to +14
using System.Drawing.Imaging;
using System.Runtime.InteropServices;
using Windows.Win32;
using static Interop;
using Gdi = Windows.Win32.Graphics.Gdi;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nit] Are these sorted? If not, please have these sorted (via the context menu):
image

Ditto below

@RussKie RussKie added the waiting-author-feedback The team requires more information from the author label Jul 19, 2022
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Jul 19, 2022
Copy link
Contributor

@Tanya-Solyanik Tanya-Solyanik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome back!

@lonitra lonitra merged commit 142fcf5 into dotnet:feature/win32 Jul 19, 2022
@lonitra lonitra deleted the BitBlt branch July 19, 2022 17:46
JeremyKuhne pushed a commit to JeremyKuhne/winforms that referenced this pull request Jul 26, 2022
* Convert BitBlt to cswin32 PInvoke

- Add HDC conversion to scopes
- Remove unused BeginPath
- NativeMethods.txt
RussKie pushed a commit that referenced this pull request Aug 9, 2022
* Convert BitBlt to cswin32 PInvoke

- Add HDC conversion to scopes
- Remove unused BeginPath
- NativeMethods.txt
@ghost ghost locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants