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

Option to disable integer base conversions #219

Closed
DavidP76 opened this issue Jul 21, 2017 · 5 comments
Closed

Option to disable integer base conversions #219

DavidP76 opened this issue Jul 21, 2017 · 5 comments

Comments

@DavidP76
Copy link

@DavidP76 DavidP76 commented Jul 21, 2017

This is a feature request, not bug report (please let me know if this is not the right place for this, and I apologize if so):

Please provide a configuration option to disable base conversions for integer Calc results

@polyvertex
Copy link
Member

@polyvertex polyvertex commented Jul 21, 2017

Hi and welcome! No worries you are in the right place.
Is there any particular reason behind this request?

@DavidP76
Copy link
Author

@DavidP76 DavidP76 commented Jul 21, 2017

The reason for this request is that the calculations that I frequently perform are very simple business calculations, and therefore I will never have the need to operate in hexidecimal or octal. It's great functionality to have available, but I would rather deactivate it for my configuration.

Thanks for the response and outstanding work on this program.

capture

@polyvertex
Copy link
Member

@polyvertex polyvertex commented Jul 21, 2017

Ok, I see no cons of adding this option. It will very probably be available in the next release.

But while I perfectly understand you would never need base conversion, the reason I asked is because I was quite surprised by the request.

  • From a purely practical standpoint the result(s) of the expression always come(s) first in the list, then the converted values so in your case it should be no bother unless I missed something?
  • From a purely technical standpoint, the CPU time needed for those conversions is insignificant

So in the end, I guess it's purely for cosmetics (visual pollution)?

@DavidP76
Copy link
Author

@DavidP76 DavidP76 commented Jul 21, 2017

Correct - simply cosmetics (just like the currency option can be removed for folks who are not working with monetary calculations). Thanks very much!

@polyvertex
Copy link
Member

@polyvertex polyvertex commented Aug 27, 2017

base_conversion setting now available in v2.15.4

@polyvertex polyvertex closed this Aug 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.