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

Concatenation of builtin functions(sincos(x)) is valid. #88

Open
alexandrustana opened this issue Aug 10, 2017 · 1 comment
Open

Concatenation of builtin functions(sincos(x)) is valid. #88

alexandrustana opened this issue Aug 10, 2017 · 1 comment

Comments

@alexandrustana
Copy link

The following expression sincos(1) is seen as valid for exp4j
Although the evaluation of the expression is done correct, meaning, even if the expression is written in the following manner sincos(1) the code evaluates it as sin(cos(1)), shouldn't the shunting yard algorithm detect that sincos(x) function is not written in a valid form and when calling the validate() method return false?

@fasseg
Copy link
Owner

fasseg commented Aug 12, 2017

Hey @alexinul, Thanks for submitting this issue. This is a bug probably due to the implicit multiplication feature.

alexandrustana added a commit to alexandrustana/exp4j that referenced this issue Aug 21, 2017
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