← Back to index | Original Bugzilla link

Bug 16339 – insertInPlace should be tested for unions and classes

Status: RESOLVED
Resolution: FIXED
Severity: trivial
Priority: P1
Component: phobos
Product: D
Version: D2
Platform: x86_64
OS: Linux
Creation time: 2016-07-30T17:18:14Z
Last change time: 2021-07-30T15:40:10Z
Keywords: pull, trivial
Assigned to: No Owner
Creator: Seb

Comments

Comment #0 by greensunny12 — 2016-07-30T17:18:14Z

It does quite a bit of manual memory manipulation, hence we should definitely test that.

Comment #1 by ilyayaroshenko — 2017-09-18T02:44:55Z

Old PR https://github.com/dlang/phobos/pull/4643 requires to be recreated

Comment #2 by ilyayaroshenko — 2017-09-18T02:46:28Z

*** Issue 16314 has been marked as a duplicate of this issue. ***

Comment #3 by dlang-bot — 2021-07-30T14:58:04Z

@lucica28 updated dlang/phobos pull request #8181 "[DSSv3] Fix issue - 16339" fixing this issue: - Fix Issue 16339 - insertInPlace should be tested for unions and classes https://github.com/dlang/phobos/pull/8181

Comment #4 by dlang-bot — 2021-07-30T15:40:10Z

dlang/phobos pull request #8181 "[DSSv3] Fix issue - 16339" was merged into master: - 2d72b54d45684e0de8f2bb566a8f5de64e238a9b by Lucian: Fix Issue 16339 - insertInPlace should be tested for unions and classes https://github.com/dlang/phobos/pull/8181