search

james91b / ida_ipython

An IDA Pro Plugin for embedding an IPython Kernel
MIT License
252 stars 46 forks source link

IDA IPython no longer crashes if IDAPython is not present. Closes #22. #24

Closed tmr232 closed 8 years ago

tmr232 commented 8 years ago

In addition, the warning messages are now prefixed to indicate they come from IDA IPython.

james91b commented 8 years ago

Nice work! It looks really good. Another way of doing the plugin check that i was looking at was using the load_plugin api. It might be a bit cleaner because you will only need one check. In addition, it would also deal with case when the plugins are loaded out of order.

tmr232 commented 8 years ago

For some reason, this does work in out-of-order scenarios. But using load_plugin seems be a better way to go.

james91b commented 8 years ago

Included in #25