D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 21103 - isDynamicArray instantiates unecessary templates
Summary: isDynamicArray instantiates unecessary templates
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:
Blocks:
 
Reported: 2020-08-02 18:14 UTC by moonlightsentinel
Modified: 2021-01-22 17:27 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 moonlightsentinel 2020-08-02 18:14:56 UTC 
See https://github.com/dlang/dmd/pull/9014#issuecomment-667704187 for an extensive overview.

TL:DR.
isDynamicArray should use an is-Expression instead of additional templates.
Comment 1 Dlang Bot 2020-08-02 18:16:03 UTC 
@MoonlightSentinel created dlang/phobos pull request #7574 "Fix Issue 21103 - isDynamicArray instantiates unecessary templates" fixing this issue:

- Fix Issue 21103 - isDynamicArray instantiates unecessary templates

https://github.com/dlang/phobos/pull/7574
Comment 2 Dlang Bot 2021-01-22 17:27:32 UTC 
dlang/phobos pull request #7574 "Fix Issue 21103 - isDynamicArray instantiates unecessary templates" was merged into master:

- 7bea4c78b90c4ca182298ebc758a41edc7227315 by MoonlightSentinel:
  Fix Issue 21103 - isDynamicArray instantiates unecessary templates
  
  Use an `is(...)` expression to do the actual check but include a
  workaround to reatin the old (buggy) behaviour regarding enums (for now).

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