Skip to content

Fixed win32 gui class management#158

Merged
dschep merged 1 commit into
dschep:masterfrom
henryx:master
Nov 8, 2017
Merged

Fixed win32 gui class management#158
dschep merged 1 commit into
dschep:masterfrom
henryx:master

Conversation

@henryx
Copy link
Copy Markdown
Contributor

@henryx henryx commented Nov 8, 2017

Using as library, it appears this problem:

import ntfy.backends.win32
ntfy.backends.win32.notify("test message", "test")
ntfy.backends.win32.notify("test message", "test2")

Second notify return this error:

  File "C:\Python27\lib\site-packages\ntfy\backends\win32.py", line 28, in __init__
    classAtom = win32gui.RegisterClass(wc)
pywintypes.error: (1410, 'RegisterClass', 'La classe esiste gi\xe0.')

This pull request (tested on Windows 10) fixes the problem

…n error because class results already registered)
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 84.452% when pulling 551c115 on henryx:master into 680891b on dschep:master.

1 similar comment
@coveralls
Copy link
Copy Markdown

coveralls commented Nov 8, 2017

Coverage Status

Coverage increased (+0.03%) to 84.452% when pulling 551c115 on henryx:master into 680891b on dschep:master.

@dschep
Copy link
Copy Markdown
Owner

dschep commented Nov 8, 2017

Thanks! LGTM :shipit:

@dschep dschep merged commit e585042 into dschep:master Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants