-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Object reference not set to an instance of an object #17
Comments
The actual traceback:
|
Can you try this ? In a terminal: From python prompt: ldtp.selectitem(...) You can find the traces on first console why it failed. Thanks |
Log ends with:
So although the rest of the log did not immediately tell me anything this points to Combobox.cs, the only commit of the three above that touches it is ef33a26, and indeed if I revert that single commit it works again. |
I think I see the issue, that commit used |
Great thanks checking. |
Fix selectitem(...). Fixing issue #17.
By the way is there any plan on adding a new binary release ? The last one seem to be from Nov 2013. |
And unfortunately it turns out my patch above did not really solve this problem - I must have missed something since the problem reappeared now. I tried to reproduce again, and it was definitely the introduced in the earlier mentioned commit. Further trying to narrow it down showed that it was a LogMessage call this time that seemed to cause this:
it also happens if I replace that log call with anything that use Any ideas ? |
I'm getting this error while using getcombovalue method. Has this been resolved? Or am I pointing to the wrong binary? Thanks for the help |
While working on #15 I noticed the error in the title of this issue when I built from master. The latest binary using installer 4.0.0 did not show that error.
This happens when I do a call to:
ldtp.selectitem(...)
.I ran a bisect and got this:
There are only 'skip'ped commits left to test.
The first bad commit could be any of:
3a912d3
ef33a26
99732f5
We cannot bisect more!
Two of those commits didn't build that's why it only narrowed it down to three.
The text was updated successfully, but these errors were encountered: