fix(memory) free classic BT memory to heap

[skandragon]

This patch releases the classic (non-BLE) bluetooth memory, which seems to fix #6. With this change, I've had the scanner running for about 20 minutes so far without a single timeout, although I still see this message at the end of a scan cycle:

E (425710) BT_HCI: btu_hcif_hdl_command_complete opcode 0x200b status 0x12

[skandragon]

I take it you're not sure if this is the proper fix? I admit I'm not sure either, but found this gem while seeking a fix, and so far my nodes have been up for just under a week without a single restart.

[rcaelers]

This call is already done in BLEScanner.cpp. I wanted to check whether it makes any difference to call it earlier. But haven't found time to do so.

You have added the call after "new Main". The Main class starts a new thread which initialises bluetooth. So the call you added may happen in parallel with the existing call. Which may cause issue during startup.

[skandragon]

Hmm, I'll look into this. It should probably free resources before any bluetooth items are in use, since I suspect there won't be any locking internally.

I'll see about making a better patch.