Cropdev / CropDev

15 stars 17 forks source link

Update masternode.cpp #7

Open dkrym opened 6 years ago

dkrym commented 6 years ago

Fix for correct last paid timestamp

dkrym commented 6 years ago

Unfortunately I am not sure how to test it, I compiled it and it seems it shows correct time (or at least different time for nodes): screenshot from 2018-03-12 11-21-29

But it seems it doesn't fix the winners list :(

dkrym commented 6 years ago

I am sorry but are you sure it is as you wrote? I am new to this so not gonna argue but my understanding is winners is list of people who won so it shouldn't be used when selecting new winner. I thought you sort people by LastPaid and from top 10% you select someone. His LastPaid stamp is updated and he is moved to the end of queue.

Anyway thanks for your input, it is very useful to understand this.

Edit: Sorry you talking about "vWinning list" now I understand, cheers

dkrym commented 6 years ago

I unfortunately don't have knowledge to create test net. Do you know what codebase is used? I am just wondering why Crop doesn't use new codebase where it is fixed