Author ronaldoussoren
Recipients paul.moore, ronaldoussoren, steve.dower, terry.reedy, tim.golden, zach.ware
Date 2018-05-29.15:01:58
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1527606118.99.0.682650639539.issue33656@psf.upfronthosting.co.za>
In-reply-to
Content
I'm not a Windows expert, but looking that the API description [1] the patch is not entirely correct. 

The API takes an integer with 3 valid values, the patch effectively passes the value 1 (which means the application claims to be DPI aware, but won't scale itself when the DPI changes for some reason).  

The API can fail, but that shouldn't cause an exception as ctypes won't automaticly convert error return values to exceptions.

P.S. The pull request seems to contain an unrelated change as well.

[1] https://msdn.microsoft.com/en-us/library/windows/desktop/dn302122(v=vs.85).aspx
History
Date User Action Args
2018-05-29 15:01:59ronaldoussorensetrecipients: + ronaldoussoren, terry.reedy, paul.moore, tim.golden, zach.ware, steve.dower
2018-05-29 15:01:58ronaldoussorensetmessageid: <1527606118.99.0.682650639539.issue33656@psf.upfronthosting.co.za>
2018-05-29 15:01:58ronaldoussorenlinkissue33656 messages
2018-05-29 15:01:58ronaldoussorencreate