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

Support for 2.12.1, overriding instead of copy/pasting #93

Open
adriaanm opened this issue Nov 8, 2016 · 3 comments
Open

Support for 2.12.1, overriding instead of copy/pasting #93

adriaanm opened this issue Nov 8, 2016 · 3 comments

Comments

@adriaanm
Copy link
Contributor

adriaanm commented Nov 8, 2016

The bad news: I broke the plugin in scala/scala#5469

The good news: there's now a hook method that can be overridden without copy/pasting from the compiler code base. Here's a sketch (didn't check javadoc output, but it works for akka): https://github.com/typesafehub/genjavadoc/compare/api-doc-comment?expand=1

The bad news: the API change with the hook method is not in 2.12.0 (it's in 2.12.1-10c609e-nightly, available from "release-temp" at "https://scala-ci.typesafe.com/artifactory/scala-release-temp/")

@adriaanm
Copy link
Contributor Author

adriaanm commented Nov 8, 2016

Pragmatically, I should probably make the method non-final again, but at least the code here then needs to be re-copy/pasted... (And we risk regressing again, because they were supposed to be final so that they were tail recursive..)

@adriaanm
Copy link
Contributor Author

adriaanm commented Nov 8, 2016

Seems like that patch fixes the plugin on the latest 2.12.1 nightly https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/924/consoleFull. Didn't check the actual doc output though

@adriaanm adriaanm changed the title Overriding versus copy/paste Support for 2.12.1, overriding instead of copy/pasting Nov 8, 2016
@lrytz
Copy link
Contributor

lrytz commented Nov 9, 2016

And we risk regressing again, because they were supposed to be final so that they were tail recursive..

you could make the method non-final, and have it forward to a separate, final, tail-recursive implementation method

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants