Skip to content

Conversation

@PastaPastaPasta
Copy link
Member

@PastaPastaPasta PastaPastaPasta commented May 12, 2019

Decided to break this off the main set since it should have been in like pr2 and I missed it. I could cherry-pick this into pr10 if wanted

b651270 util: Throw tinyformat::format_error on formatting error (Wladimir J. van der Laan)
3b092bd util: Properly handle errors during log message formatting (Wladimir J. van der Laan)

Tree-SHA512: 85e3b7afec2255fc88034187f1abd6060e9421de17ed4e3d918416f393429a99cc2c974b362099aaaff6970549df47664bea4c857c4e46acc0789663201dc541

@UdjinM6
Copy link

UdjinM6 commented May 15, 2019

llmq/quorums.cpp:215:21: note: in expansion of macro 'LogPrint'
                     LogPrint("llmq", debugMsg);

https://travis-ci.org/dashpay/dash/jobs/531315434#L2159

@codablock codablock added this to the 14.1 milestone May 16, 2019
laanwj and others added 2 commits May 16, 2019 12:41
…ormatting

b651270 util: Throw tinyformat::format_error on formatting error (Wladimir J. van der Laan)
3b092bd util: Properly handle errors during log message formatting (Wladimir J. van der Laan)

Tree-SHA512: 85e3b7afec2255fc88034187f1abd6060e9421de17ed4e3d918416f393429a99cc2c974b362099aaaff6970549df47664bea4c857c4e46acc0789663201dc541
Signed-off-by: Pasta <[email protected]>

"cast" debugMsg to a c string pt 2

Signed-off-by: Pasta <[email protected]>
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@codablock codablock merged commit a6eee07 into dashpay:develop May 21, 2019
@PastaPastaPasta PastaPastaPasta deleted the backport-9963 branch May 21, 2019 13:56
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.

4 participants