Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Ceylon logging should support loggers with class name #695

Open
dlkw opened this issue Oct 25, 2017 · 1 comment
Open

Ceylon logging should support loggers with class name #695

dlkw opened this issue Oct 25, 2017 · 1 comment

Comments

@dlkw
Copy link
Contributor

dlkw commented Oct 25, 2017

The ceylon.logging::logger function accepts an argument with type

shared alias Category => Module|Package;

I suggest adding ceylon.language.meta.declaration::ClassDeclaration to the accepted argument types, so a logger can have a class-specific name (as is usual in Java) when created as

Logger log = logger(`class`);

inside a class or member class. Not having class-specific names for loggers makes reading the log more difficult: only the module or even package name is rather coarse to get an idea where the log entry was produced.

@gavinking
Copy link
Contributor

You're welcome to propose a patch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants