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.

Sign up for GitHub

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

Jump to bottom

impl ToHtml not accepted as a template argument #57

Closed
kornelski opened this issue Aug 19, 2019 · 0 comments · Fixed by #59
Closed

impl ToHtml not accepted as a template argument #57

kornelski opened this issue Aug 19, 2019 · 0 comments · Fixed by #59

Comments

@kornelski
Copy link
Contributor

This is accepted:

@(head: &ToHtml, content: &ToHtml)

but this is not:

@(head: impl ToHtml, content: impl ToHtml)
@kornelski kornelski mentioned this issue Aug 19, 2019
Closed
kaj added a commit that referenced this issue Aug 23, 2019
@kaj
Accept explicit impl and dyn in types.
9cc30de 
This should fix #57.
@kaj kaj mentioned this issue Aug 23, 2019
Merged
@kaj kaj closed this as completed in #59 Aug 23, 2019
kaj added a commit that referenced this issue Aug 28, 2019
@kaj
Release 0.7.2
af2a581 
* Issue #53, PR #60: Allow empty strings everywhere quoted strings are
  allowed.
* Issue #57, PR #59:Accept explicit impl and dyn in types.
* Relax over-strict whitespace requirements, fix a regression in 0.7.0.
* PR #56: Require buf reference to implement Write, not buf itself
* PR #58: Fix warnings in generated code.
* Update bytecount to 0.6.0.
* Remove no-longer-used imports.

Thanks to @kornelski for multiple contributions.
kaj added a commit that referenced this issue Aug 28, 2019
@kaj
Release 0.7.2
2dae322 
* Issue #53, PR #60: Allow empty strings everywhere quoted strings are
  allowed.
* Issue #57, PR #59:Accept explicit impl and dyn in types.
* Relax over-strict whitespace requirements, fix a regression in 0.7.0.
* PR #56: Require buf reference to implement Write, not buf itself
* PR #58: Fix warnings in generated code.
* Remove no-longer-used imports.

Thanks to @kornelski for multiple contributions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Accept explicit impl and dyn in types. kaj/ructe
1 participant
@kornelski