Skip to content
This repository was archived by the owner on Feb 14, 2022. It is now read-only.

Optimizations#118

Merged
JacopoWolf merged 11 commits into
rel/3.1from
dev/optimization
Nov 2, 2020
Merged

Optimizations#118
JacopoWolf merged 11 commits into
rel/3.1from
dev/optimization

Conversation

@JacopoWolf
Copy link
Copy Markdown
Owner

@JacopoWolf JacopoWolf commented Oct 31, 2020

  • performance optimizations
  • added the following APIs:
    • IAsyncStackWrapper.SubmitAndGet( TIn ) that returns the submitted task
    • IAsyncStackWrapperCore.IsElaborating { get; }
  • removed the following APIs:
    • IAsyncStackWrapperCore.Submit( Task<T> ) is now private
  • closes Write in-depth tests #102
  • closes Further performance optimizations #111

- InstancesHolder is now a dictionary of linked lists for holding multiple instance in a future expansion.
- Async tests keeps breaking. [Ignore] for now
added check for parameterless constructor.
- better error reporting
- more performance improvements
- now Submit(TIn) returns the submitted task
- updated tests
- added more async tests
@JacopoWolf JacopoWolf added enhancement New feature Core Relative to InjectionCore testing Testing related labels Oct 31, 2020
@JacopoWolf JacopoWolf added this to the v3.1 milestone Oct 31, 2020
@JacopoWolf JacopoWolf self-assigned this Oct 31, 2020
@JacopoWolf JacopoWolf marked this pull request as ready for review November 2, 2020 14:56
@JacopoWolf JacopoWolf added the Async Relative to asynchronous classes label Nov 2, 2020
@JacopoWolf JacopoWolf merged commit 271dc99 into rel/3.1 Nov 2, 2020
@JacopoWolf JacopoWolf deleted the dev/optimization branch November 2, 2020 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Async Relative to asynchronous classes Core Relative to InjectionCore enhancement New feature testing Testing related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant