D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 21556 - std.container.array: insertAfter is wrong and should be tested
Summary: std.container.array: insertAfter is wrong and should be tested
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: All All
: P1 normal
Assignee: No Owner
URL:
Keywords: pull
Depends on: 21555
Blocks:
  Show dependency tree / graph
 
Reported: 2021-01-18 01:34 UTC by Luís Ferreira
Modified: 2021-01-19 01:20 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Luís Ferreira 2021-01-18 01:34:40 UTC 
Due to insertBack wrong behavior, insertAfter is also returning 0 elements everytime.

See dependent issue #21555.
Comment 1 Dlang Bot 2021-01-18 20:55:38 UTC 
@ljmf00 updated dlang/phobos pull request #7750 "Fix issue 21555, 21556 - container: array: insertAfter and insertBack are wrong" fixing this issue:

- Fix issues 21555, 21556: container: array: add missing tests for Array!T .insertBack and .insertAfter
  
  Signed-off-by: Luís Ferreira <contact@lsferreira.net>

https://github.com/dlang/phobos/pull/7750
Comment 2 Dlang Bot 2021-01-19 01:20:17 UTC 
dlang/phobos pull request #7750 "Fix issue 21555, 21556 - container: array: insertAfter and insertBack are wrong" was merged into master:

- 30f71fd5705beaa3dc9c5131b081c60ccaa0974c by Luís Ferreira:
  Fix issues 21555, 21556: container: array: add missing tests for Array!T .insertBack and .insertAfter
  
  Signed-off-by: Luís Ferreira <contact@lsferreira.net>

https://github.com/dlang/phobos/pull/7750