Opened 11 years ago

Last modified 4 years ago

#978 new defect/bug

Use GetWindowLongPtr and GWLP_HINSTANCE.

Reported by: kgbricola Owned by:
Priority: trivial Milestone: version 0.6.0
Component: port/windows Version: git master
Severity: Keywords: windows, win32, api, code quality
Cc: cp15


The attached patch removes the deprecated usage of GWL_HINSTANCE and GetWindowLong? for win32.

Attachments (1)

getwindowlongptr.patch (3.7 KB) - added by kgbricola 11 years ago.

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by kgbricola

comment:1 Changed 11 years ago by sleske

Hey, the win32 GUI is alive :-). There have been no substantial patches since December 2009...

comment:2 Changed 11 years ago by…

Well, the question is, what needs to be done to get the patch in? Does it need some tests if it breaks something? Is there needed someone to review it? Has anyone an opinion about it, what needs to be improved in the patch before it could be accepted? Is the patch in the wrong format?

comment:3 Changed 11 years ago by tegzed

Hi kgbricola, I'm not using the windows port, but I think some feedback with different win32 toolchains (mingw32, different msvc versions) would be useful.

comment:4 Changed 10 years ago by usul

  • Cc cp15 added
  • Keywords windows win32 api code quality added
  • Milestone set to version 0.5.1

Obviously this patch hadn't been applied and is waiting for feedback. I'm not sure if we will still support the win32 GUI because currently we have very limited resources :/

comment:5 Changed 10 years ago by sleske

Well, I see no problem with supporting the Win32 GUI - if there's someone willing to work on it. Maybe the OP is interested?

I can apply the patch; I just need someone who can build on Windows to test it for me.

comment:6 Changed 5 years ago by

  • Milestone changed from version 0.5.1 to version 0.5.2

This ticket was pushed back in order to bring 0.5.1 out soon.

comment:7 Changed 4 years ago by

  • Milestone changed from version 0.5.2 to version 0.6.0

Ticket retargeted after milestone closed

Note: See TracTickets for help on using tickets.