Harshit-Bhimsaria / dragonfly

Automatically exported from code.google.com/p/dragonfly
0 stars 0 forks source link

Invaild int check in 64-bit windows #16

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Run 64-bit windows.
Install 64-bit python (I used Canopy).
Install dragonfly.
Run the attached example code.

I got an assert error.  The culprit is that 64 bit windows seems to be sending 
a handle of type 'long' instead of the expected type 'int'.

The fix is to change windows.py to read:
    def _set_handle(self, handle):
        assert isinstance(handle, int) or isinstance(handle, long)
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^

Original issue reported on code.google.com by nathanie...@gmail.com on 25 Dec 2013 at 11:30

Attachments:

GoogleCodeExporter commented 8 years ago
The assertion error:

Traceback (most recent call last):
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\win3
2com\server\policy.py", line 277, in _Invoke_
    return self._invoke_(dispid, lcid, wFlags, args)
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\win3
2com\server\policy.py", line 282, in _invoke_
    return S_OK, -1, self._invokeex_(dispid, lcid, wFlags, args, None, None)
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\win3
2com\server\policy.py", line 585, in _invokeex_
    return func(*args)
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\drag
onfly\engines\engine_sapi5.py", line 195, in phrase_start_callback
    window = Window.get_foreground()
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\drag
onfly\windows\window.py", line 52, in get_foreground
    window = Window(handle=handle)
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\drag
onfly\windows\window.py", line 77, in __init__
    self.handle = handle
  File "C:\Users\than\AppData\Local\Enthought\Canopy\User\lib\site-packages\drag
onfly\windows\window.py", line 88, in _set_handle
    assert isinstance(handle, int)
AssertionError

Original comment by nathanie...@gmail.com on 25 Dec 2013 at 11:32

GoogleCodeExporter commented 8 years ago
The Dragonfly project has moved to GitHub here:
https://github.com/t4ngo/dragonfly

Tracking of this issue specifically has moved here:
https://github.com/t4ngo/dragonfly/issues/2

Original comment by ct.butcher@gmail.com on 14 Sep 2014 at 12:02