search

larpon / QtFirebase

An effort to bring Google's Firebase C++ API to Qt + QML
MIT License
284 stars 83 forks source link

Remote config init #104

Closed MasterMind2k closed 5 years ago

MasterMind2k commented 5 years ago

Fix for #103.

Module initializer object got destructed before the post-processing code was able to process the Future, resulting in invalid state.

I took the liberty to remove some compiler warnings on implicit casts. Also, I removed duplicated calls on the key when copying into default byte array.

Also, I'm not sure if __defaultsByteArrayList needs to be a class member?

Hope this helps.

larpon commented 5 years ago

@MasterMind2k - Thanks! Looks good!