Skip to content
This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Build optiomization fork #274

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Build optiomization fork #274

wants to merge 19 commits into from

Conversation

aliak00
Copy link
Contributor

@aliak00 aliak00 commented Aug 6, 2019

Let's see how this goes...

@aliak00
Copy link
Contributor Author

aliak00 commented Aug 6, 2019

😡

Looked like there was a decent amount of unsynchronized access of
data in the tests. Typical things included declaring arrays on
the main thread and appending to them in a callback (that's not
guaranteed to land on the main thread)

And then the Testing Networking Proxy also had a slight bug that
stored the call data after the completion callback was given to
the client. So I divided the calldata in to request data that is
guaranteed to be there before the client completion block is called
but after the networking proxy actually makes the call, and a
responses object which is separate an only available upon return
of the callback
@aliak00 aliak00 force-pushed the build-optiomization-fork branch from 7ab694e to 3ee4483 Compare August 12, 2019 23:26
aliak00 and others added 4 commits August 13, 2019 02:23
For some silly reason, a task manager was for a user inside
the protocol and stored in a map, but the idiot who wrote it
didn't realize that user already has a taskManager so there's
no reason to do that.

Also the executor in the TaskOperation had a redundant done
callback that just set isFinished to true immdiately after
going in the execution block. So removed that
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants