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

Added support for JEE JSP tags. #18

Closed
wants to merge 3 commits into from
Closed

Added support for JEE JSP tags. #18

wants to merge 3 commits into from

Conversation

leogtzr
Copy link
Contributor

@leogtzr leogtzr commented Oct 16, 2014

Added support for JSP tags (please check the tld file and the Exp4JTagHandler.

<expr4j:expr expr="sin(x)" x="4">
exp4j tags

    </expr4j:expr>

    <expr4j:expr expr="sin(x)" x="4" />

<expr4j:expr x="2">
    <jsp:attribute name="expr">
       2  + x^2
    </jsp:attribute>
</expr4j:expr>

@leogtzr
Copy link
Contributor Author

leogtzr commented Oct 16, 2014

I will write the code to pack the tld file into a jar file tomorrow.

@fasseg
Copy link
Owner

fasseg commented Oct 19, 2014

Hey @leogtzr,

I think this would make an excellent project on it's own, maybe exp4j-tld or something, but IMHO it's out of scope for the exp4j core. Please let me know if you're interested in setting up such a project, so I can put up some links.

@fasseg fasseg closed this Oct 19, 2014
@leogtzr
Copy link
Contributor Author

leogtzr commented Oct 19, 2014

Yes, I am interested! @fasseg please go ahead

@fasseg
Copy link
Owner

fasseg commented Oct 20, 2014

Very cool!
What I meant was that you create a project on e.g. GitHub. Once it's finished I will link to your project from the exp4j pages and mention it in the docs.

@leogtzr
Copy link
Contributor Author

leogtzr commented Dec 6, 2014

Done!
https://github.com/leogtzr/exp4j_tld

Thanks

@leogtzr
Copy link
Contributor Author

leogtzr commented Dec 8, 2014

Added support for custom functions :)

@fasseg
Copy link
Owner

fasseg commented Dec 8, 2014

Very cool I will update the docs and link to your project.

@fasseg
Copy link
Owner

fasseg commented Dec 8, 2014

I released 0.4.3 and added a link.

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