-
Notifications
You must be signed in to change notification settings - Fork 40
use fileline in logs in developer mode instead of funcion name #1525
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
Conversation
could you show me what an example log would look like in dev mode after your changes? |
src/utils/utils_log.h
Outdated
#ifdef UMF_DEVELOPER_MODE | ||
#define UMF_STRINGIFY(x) #x | ||
#define UMF_TOSTRING(x) UMF_STRINGIFY(x) | ||
#define UMF_FUNC_DESC() __FILE__ ":" UMF_TOSTRING(__LINE__) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you change this to "file: line (func)" or sth similar that looks ok? I mean I would still like to know the name of the function from which the log was printed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
applied
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please show me what the current log looks like
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please show me what the current log looks like
[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:82 umfInit: UMF tracker created
[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:92 umfInit: UMF IPC cache initialized
[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:99 umfInit: UMF library initialized
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be squashed
No description provided.