Skip to content

Task, Queues and Multithreading#153

Merged
Dandielo merged 29 commits intomasterfrom
ms-2303010415
May 13, 2023
Merged

Task, Queues and Multithreading#153
Dandielo merged 29 commits intomasterfrom
ms-2303010415

Conversation

@Dandielo
Copy link
Copy Markdown
Member

@Dandielo Dandielo commented May 6, 2023

No description provided.

* Added 'any', 'count' to HashMap containers
* Implemented 'try_get' for mutable HashMap containers
* Implemented merge sort for linked list nodes `AtomicLinkedList`
* The utf8-to-wide utility functions are now available publicly.
* Updated code in affected projects.
* This includes the move to Conan v2
* Returns valid pointer from a 'consteval' created object.
* added 'wait_for' and 'wait_for_all' and their 'manual_' versions.
* updated all affected source code.
* added 'schedule_on' and variants.
* added 'resume_on' and variants.
@Dandielo Dandielo added tech-debt Technical debt that needs to be addressed, but can be tolerated for some time enhancement Improvements or additions to code labels May 6, 2023
Dandielo added 8 commits May 7, 2023 13:47
* Still keeping /O1 as the main workaround
* Waiting for next VS 2022 release.
* Split iceshard core entry points into separate stages.
* Renamed 'ManualResetSemaphore' to 'ManualResetBarrier'
* Removed old custom types.
@Dandielo Dandielo self-assigned this May 13, 2023
@Dandielo Dandielo marked this pull request as ready for review May 13, 2023 04:33
@Dandielo
Copy link
Copy Markdown
Member Author

Finishing up the milestone.

Comment thread source/code/core/tasks/private/sync_manual_events.cxx
Comment thread source/code/core/utils/public/ice/result_codes.hxx
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create a ticket to properly integrate Chipmunk2d

{
for (ice::AssetEntry* entry : _asset_resources)
{
//if (entry->refcount.load(std::memory_order_relaxed) > 0)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create task to check for proper asset releasing or at least warnings in dev-ui

Comment thread source/code/systems/resource_system/private/resource_tracker.cxx
ice::AssetType type,
ice::String name,
ice::String name
) noexcept -> ice::Asset
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create a brainstorming task to see if the 'bind' method of asset system can be changed

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created issue: #159

@Dandielo Dandielo merged commit 666b521 into master May 13, 2023
@Dandielo Dandielo deleted the ms-2303010415 branch October 25, 2023 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improvements or additions to code tech-debt Technical debt that needs to be addressed, but can be tolerated for some time

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant