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

Change "XMR" to global.config.general.coinCode in coin javascript #16

Closed
bobbieltd opened this issue Feb 27, 2018 · 7 comments
Closed

Comments

@bobbieltd
Copy link

In coin javascript, there are alot of hardcoded XMR log/email/text (mainly from Snipa's toriginal code), can you change all "XMR" to global.config.general.coinCode ?
I know it's not important. Just in case, you want to do it ;), great job !

@MoneroOcean
Copy link
Owner

OK. Will do it this week.

@MoneroOcean
Copy link
Owner

MoneroOcean commented Mar 3, 2018

I must be blind but the only XMR hardcore in common code I found and fixed is here:

e45c946

@bobbieltd
Copy link
Author

Never mind. It's minor issue, I can do by hand to change. For orphan leaf node, here is Venthos' solution. We only need to wait for block height pass that orphan block (+1), if blockManager still can't find by hash, we can store as orphan block (no need timer check) : Venthos@4dd98c1

@bobbieltd
Copy link
Author

One more things, for pool module, he doesn't use updatetemplate every 300ms but he use getlasblockheader to check. It's less CPU usage --> more effective. What do you think ?

@MoneroOcean
Copy link
Owner

MoneroOcean commented Mar 3, 2018

I think you should check my code. I use getlastblockheader for ages)

@MoneroOcean
Copy link
Owner

Ventos fix is simpler, but I would like to see how Snipa will react to it. There can be a problem if master node has block on that height already that will be actually orphaned (and not our block from leaf node). In this case we will end with wrong block reward.

@bobbieltd
Copy link
Author

Not neccessary anymore. Close this issue.

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