summaryrefslogtreecommitdiff
path: root/include/mbgl/actor
Commit message (Collapse)AuthorAgeFilesLines
* [core] Avoid blocking in Thread<Object> constructor (#12151)Anand Thakker2018-07-035-48/+179
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Introduce AspiringActor, EstablishedActor This pair of objects represents the two-phase (parent-thread / child-thread) construction that's needed to support constructing Thread<Object> without blocking until the child thread is up and running. An `AspiringActor<O>` is responsible for: - ownership of the actor's `Mailbox` - allocating the memory for (but *not* constructing) the target object `O` Using these two pieces--the mailbox and a stable address for `O`--an `AspiringActor<O>` can accept messages for the target object, or provide `ActorRef<O>`s that do so, before the object has actually been constructed by the corresponding `EstablishedActor<O>`. (Such messages are queued in the mailbox until after the object is constructed.) This allows for an `AspiringActor<O>` to be created and safely used by a thread other than the one on which the target object will (eventually) live. An `EstablishedActor<O>` is responsible for managing the lifetime of the target object `O` and the open/closed state of the parent's `mailbox`. The `O` object's lifetime is contained by that of its owning `EstablishedActor<O>`: the `EstablishedActor` constructor executes the `O` constructor via "placement new", constructing it at the address provided by the parent `AspiringActor`, and the `~EstablishedActor` destructor similarly executes the `~O` destructor (after closing the mailbox). `EstablishedActor` should therefore live entirely on the thread intended to own `O`. * Remove Actor#{invoke,ask}
* [core] Actor/ActorRef - ask calls to void methodsIvo van Dongen2017-09-221-0/+26
| | | | - allows to wait for execution of void methods as well as non-void methods
* [core] current schedulerIvo van Dongen2017-08-091-6/+9
| | | | - Adds a way to set the current scheduler on the thread to be used whenever a mailbox is created that needs to reply on this thread
* [core] make actor self reference optional - againIvo van Dongen2017-08-081-2/+7
| | | | - uses a different method of constructor selection that also works on Apple clang < 8.2.
* [core] Removed unused constructorupstream/tvn-test-ci-time-outThiago Marcos P. Santos2017-08-071-8/+0
| | | | | | | | | | Not in use at the moment, was causing a compiler error on Apple LLVM version 8.0.0 (clang-800.0.42.1). ``` src/mbgl/sprite/sprite_loader.cpp:25:11: error: call to constructor of 'Actor<mbgl::SpriteLoaderWorker>' is ambiguous ```
* [core] make actor self reference optionalIvo van Dongen2017-07-241-3/+15
|
* [core] add ask pattern to actor refIvo van Dongen2017-07-241-0/+21
|
* [core] implement ask pattern in actorIvo van Dongen2017-07-242-0/+46
|
* [core] Store pointer instead of referect to the Object in ActorRefThiago Marcos P. Santos2017-06-261-3/+3
| | | | | | | | | | | GCC complains about implicitly deleted assigment constructor when trying to do this: void setResourceTransform(optional<ActorRef<ResourceTransform>> transform) { resourceTransform = transform; } optional<ActorRef<ResourceTransform>> resourceTransform;
* [core] Make the mbgl/actor headers publicThiago Marcos P. Santos2017-06-263-0/+168
| | | | | They will be needed by the DefaultFileSource, something that we also export as public.
* [core] allow self closing mailbox/actorIvo van Dongen2017-05-271-1/+1
|
* [core] Block in Mailbox::close() until neither receive nor push are in progressJohn Firebaugh2017-05-241-2/+4
| | | | | | | | | | | | | | Otherwise, an ActorRef that's in the process of sending a message could attempt to access an invalid Scheduler reference: Thread 1 Thread 2 -------------------------------------------------- Scheduler::Scheduler Actor::Actor weakMailbox.lock() Actor::~Actor Scheduler::~Scheduler mailbox->push() scheduler.schedule() 💣
* [core] Move actor/{mailbox,scheduler}.hpp to public include directoryJohn Firebaugh2017-03-282-0/+71
Map constructor takes Scheduler&, and consumers are expected to define an implementation. Therefore the interface must be public.