epam / epam/Indigo

Replace ObjArray with std::vector

Open
#815 0 comments 0 reactions 1 assignee Claimed by @AStepaniuk View on GitHub
Dominant language
C++
Stars
406
Forks
134
Avg merge
2d 11h
Merged PRs (30d)
24

Description

ObjArray is an object-wise wrapper around Array.
Though ObjArray is still not safe to use with C++ objects because underlying Array does not properly copies/moves objects on buffer re-allocation.

A simple solution here: replace ObjArray with std::vector, which has pretty similar interface and correctly deals with contained objects.
Challenges:
1. The contained type needs to have move constructor, marked as noexcept
2. ObjArray can `qsort()` items using its own method. This method needs to be extracted as a standalone function, implemented as an algorithm

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.