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

Centralize the logging code into its own folder in Common. #465

Merged
merged 1 commit into from Jun 26, 2014

Conversation

lioncash
Copy link
Member

@lioncash lioncash commented Jun 5, 2014

Since it's one of the more important things in Commons (literally ubiquitous throughout the project) I think it should have it's own folder to separate logging code from the miscellaneous utility files we have in Common.

#include "Common/Thread.h"
#include "Common/Logging/LogManager.h"

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@neobrain
Copy link
Member

LGTM.

@dolphin-emu-bot rebuild

@neobrain
Copy link
Member

Build failed apparently, please rebase :/

@lioncash
Copy link
Member Author

Will do when I get home.
On Jun 13, 2014 3:06 PM, "Tony Wasserka" notifications@github.com wrote:

Build failed apparently, please rebase :/


Reply to this email directly or view it on GitHub
#465 (comment).

@neobrain
Copy link
Member

@dolphin-emu-bot rebuild

lioncash added a commit that referenced this pull request Jun 26, 2014
Centralize the logging code into its own folder in Common.
@lioncash lioncash merged commit a5aac68 into dolphin-emu:master Jun 26, 2014
@lioncash lioncash deleted the logging branch June 26, 2014 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants