Hello Mirek,


Quote:

ReadIcon crashed on png. But it does not anymore when I am testing. Can you check this one?

I've just synced my copy of trunk and recompiled TheIDE (on Linux, in both DEBUG and RELEASE modes). The issue seems to be resolved. The search manager downloaded and displayed the icons correctly. Thanks!


By the way, some more observations on the new code:

1. It does not discard empty entries in json file (while reading). Empty entries can show up in both menu and in the search manager. The
original patch prevented this. A cosmetics issue really, but looks ugly when it happens.

2. Default Google search is not available when the list has at least one entry. (The original patch prevented this too by allowing the user to unset the default engine) The only way to reset the default search engine to Google is by deleting the entire list.

4. ArrayCtrl has visible vertical and horizontal grids, which seems unnecessary as it uses even/odd line coloring already.

Thank you for your efforts.