Issue 7152 - Can't assign null to default argument #1448

Merged
merged 1 commit into from Jan 10, 2013

Conversation

Projects
None yet
3 participants
Member

9rnsr commented Jan 9, 2013

http://d.puremagic.com/issues/show_bug.cgi?id=7152

This had not implemented as a part of issue 7019.

fix Issue 7152 - Can't assign null to default argument
This had not implemented as a part of issue 7019.
Member

yebblies commented Jan 9, 2013

This test case does not contain a null default arg, was this intentional? Based on the numbering I assume it's from 7019, which is still open. Not sure what's going on there.

Member

9rnsr commented Jan 9, 2013

I simply uses S7019 for the test case of issue 7152.

The point of issue 7152 is:
void foo(S7019 s = 5) { ... }
should be automatically translated to:
void foo(S7019 s = S7019(5))

So, it is intentional.

I assume it's from 7019, which is still open.

Oh, I didn't noticed that is still open. Hmm, after merging this, we will be able to close both 7019 and 7152.

WalterBright added a commit that referenced this pull request Jan 10, 2013

Merge pull request #1448 from 9rnsr/fix7152
Issue 7152 - Can't assign null to default argument

@WalterBright WalterBright merged commit c7215bc into dlang:master Jan 10, 2013

1 check passed

default Pass: 10
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment