Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

log: add wlr_ prefix #1011

Closed
wants to merge 2 commits into from
Closed

log: add wlr_ prefix #1011

wants to merge 2 commits into from

Conversation

emersion
Copy link
Member

See swaywm/sway#2009

Not sure what to do with L_ERROR et al.

@ddevault
Copy link
Contributor

How about s/L_DEBUG/WLR_DEBUG/g et al?

@emersion
Copy link
Member Author

Yeah, done that, works for me.

cc @Ongy @Timidger: the whole logging API has been renamed.

@ddevault
Copy link
Contributor

Does #2009 need to be updated?

@emersion
Copy link
Member Author

Does #2009 need to be updated?

Yeah, I bet it won't work anymore, because it redefines wlr_log_init. It also needs to setup wlroots to use the sway logging callback.

@emersion emersion mentioned this pull request Jul 1, 2018
95 tasks
@emersion emersion closed this Jul 6, 2018
@emersion
Copy link
Member Author

emersion commented Jul 6, 2018

This kind of PRs needs to be merged pretty quickly otherwise there are tons of conflicts. Will resend another one.

@emersion emersion deleted the prefix-log branch July 7, 2018 08:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants