Skip to content

Conversation

@pniewiejski
Copy link

Hi,

In the latest release of plantuml (V1.2020.7), if a function does not have a return directive we will get an error.
See changes at plantuml/plantuml@9077d97, in the CodeIteratorReturnFunction.java

I've changed every function declaration to procedure where there is nothing returned.

@adrianvlupu
Copy link
Owner

Oh, thanks. I will merge this. Sorry for not responding, had a lot of stuff to do.

@adrianvlupu
Copy link
Owner

Great, now everything breaks on 1.2019.09 (my local version).
Also on 1.2020.07 dynamic diagrams break. I have to find a way to merge all of this together.

@pniewiejski
Copy link
Author

Sorry to hear that 😏

I guess keeping backward compatibility is not a virtue that plantuml can be proud of 😅

adrianvlupu added a commit that referenced this pull request Apr 25, 2020
@adrianvlupu
Copy link
Owner

Yes yes, not the first time stuff breaks. Managed to get it working though, struggling with the branches now. I think I will tag commits with the plantuml version from now

@adrianvlupu adrianvlupu merged commit 1a8e291 into adrianvlupu:latest Apr 25, 2020
@pniewiejski pniewiejski deleted the pr-update-plantuml-1.2020.07 branch April 25, 2020 20:30
@pniewiejski pniewiejski restored the pr-update-plantuml-1.2020.07 branch April 25, 2020 20:30
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

Successfully merging this pull request may close these issues.

2 participants