I just landed a patch [1] that replaces all instances of "prlog.h" in gecko
code with "mozilla/Logging.h". Currently it's not terribly interesting, it just
includes "prlog.h", but in the future we're going to start swapping out
functionality.
To support this effort please use "mozilla/Logging.h" instead of "prlog.h" in
gecko code.
If you need to fix up a patch probably all that's required is:
|sed -i -E -e 's/#include\s+("|<)prlog.h("|>)/#include
"mozilla\/Logging.h"/'|
Other improvements pending [2]:
- PR_LOG => MOZ_LOG
- PR_LOG_TEST => MOZ_LOG_TEST
- Remove usage of PR_LOG_ALWAYS
- Remove usage of fake log levels (PR_LOG_DEBUG + 1, etc)
- A fancy new LogLevel enum class
For the last 3 points I will follow up in separate thread explaining the
current plan.
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1165518
[2] https://bugzilla.mozilla.org/show_bug.cgi?id=1165515
_______________________________________________
dev-platform mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-platform
