Wrappers for OCTToxAV - Githubissues

Antidote-for-Tox / objcTox

No longer maintained

Mozilla Public License 2.0

37 stars 12 forks source link

Wrappers for OCTToxAV #17

Closed Chuongv closed 9 years ago

Chuongv commented 9 years ago

Beginning of the wrappers have been started. For the start method, I just copied from OCTTox and replaced everything with ToxAV since I'm not quite sure how dispatching queues work. So if I missed anything let me know :smiley_cat:

Chuongv commented 9 years ago

Currently a problem with the OCTToxAVTest, for some reason if you run the tests separately they will pass. However if you run them sequentially together as a group, we get a TOX_MALLOC error from trying to initialize OCTTox. Which leads me to believe there might be something wrong with the teardown process.

dvor commented 9 years ago

For me tests pass. Maybe you could try to reset iOS Simulator and run tests once again?

Travis failed building tests with error. Will you take a look at it?

Chuongv commented 9 years ago

Test still fails even when resetting the iOS simulator. I'll add a stop in the dealloc for toxav and see if that changes anything.

Chuongv commented 9 years ago

For me, the malloc error doesn't come up on the iphone 6 plus or iphone 5s simulator but it does for iphone 4s and 5. Weird...

coveralls commented 9 years ago

Coverage decreased (-1.39%) to 71.21% when pulling d67e112f0c9f061c159e01378d6d4b715a597585 on Chuongv:audio into b5ced0cc6c57ff89b4938dd6c3e0ae4e4e834f96 on dvor:audio.

coveralls commented 9 years ago

Coverage decreased (-1.36%) to 71.23% when pulling 97870b6392a2f1a708f5910d7f3ad0b602c103e0 on Chuongv:audio into b5ced0cc6c57ff89b4938dd6c3e0ae4e4e834f96 on dvor:audio.

dvor commented 9 years ago

This coveralls comments are annoying.