voxeliq icon indicating copy to clipboard operation
voxeliq copied to clipboard

Published 20 hours ago • verified publisher icon bonesoul

Refactor out the Logger configuration.

Open bonesoul opened this issue 12 years ago • 3 comments

As this isn't any kind of server-like software, I don't think we need configurable loggers. So I'll be refactoring out logger configuration functionality -- and will bind the console and file log will be just statically.

bonesoul avatar Jan 15 '13 15:01 bonesoul

initial work: https://github.com/raistlinthewiz/voxeliq/commit/d36333d7785ef573033d3a180e83bc5941e63b0b

bonesoul avatar Jan 15 '13 16:01 bonesoul

I'm always loathe to remove functionality and configurable loggers are wonderful for runtime debugging.

That said the overhead may not be worth it.

Delwin9999 avatar Jan 15 '13 23:01 Delwin9999

With the latest commit, I left them as still configurable ones.

Loggers are already initialized by the sample game where modifying the game.cs code and configuring them in-code is just similar to configuring them from the ini file.

And for release builds, I don't think anyone will be willing to let their users to configure game-clients logging functionality from the ini.

bonesoul avatar Jan 16 '13 00:01 bonesoul