Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better REST automatic method implementation generation ( qualified type names + local module import ) #108

Merged
merged 2 commits into from Oct 12, 2012

Conversation

Projects
None yet
2 participants
@mihails-strasuns
Copy link
Contributor

commented Oct 11, 2012

Continuation of #104
I think it is done now. Utility templates looks a bit more monstrous now, but that is because I have made an according pull requests to phobos :
dlang/phobos#863
dlang/phobos#862
Until it is accepted ( and released ) or rejected an exact copy is included in this pull request

Dicebot added some commits Oct 7, 2012

@s-ludwig

This comment has been minimized.

Copy link
Member

commented Oct 12, 2012

Very nice. I'll merge and test on my stuff ASAP.

s-ludwig added a commit that referenced this pull request Oct 12, 2012

Merge pull request #108 from Dicebot/rest_method_gen
Better REST automatic method implementation generation ( qualified type names + local module import )

@s-ludwig s-ludwig merged commit 24ce385 into vibe-d:master Oct 12, 2012

@s-ludwig

This comment has been minimized.

Copy link
Member

commented Oct 12, 2012

Apart from some typos (fixed 367c975) everything seems to work nicely. This will make the interface stuff much more robust, thanks again!

@s-ludwig

This comment has been minimized.

Copy link
Member

commented Oct 12, 2012

Added support for enums and static arrays in 9b52c0b.

Phobos fullyQualifiedName! does not seem to work with inner types - it just outputs the unqualified name - so I disabled the corresponding unit tests for now. -> http://d.puremagic.com/issues/show_bug.cgi?id=8806

@mihails-strasuns

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2012

Yeas, that is related to dlang/phobos#826 pull request - it has been merged, but I assume you are not using master phobos version :) I ll do an update on next dmd/phobos release.

@mihails-strasuns mihails-strasuns deleted the mihails-strasuns:rest_method_gen branch Jan 17, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.