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

Prefix print functions with lf_ #1127

Merged
merged 11 commits into from May 1, 2022
Merged

Prefix print functions with lf_ #1127

merged 11 commits into from May 1, 2022

Conversation

housengw
Copy link
Contributor

@housengw housengw commented Apr 30, 2022

This PR addresses the issue #1124 and prefixes print functions in the C target (ex. info_print, debug_print, ...) with lf_.

Here is a detailed list of changes:

info_print -> lf_print
log_print -> lf_print_log
debug_print -> lf_print_debug
error_print -> lf_print_error
warning_print -> lf_print_warning
error_print_and_exit -> lf_print_error_and_exit

The deprecated names are still available currently but will be removed in a future release.

@housengw housengw requested review from cmnrd and removed request for cmnrd April 30, 2022 18:37
@housengw housengw self-assigned this Apr 30, 2022
@housengw housengw added the c Related to C target label Apr 30, 2022
@housengw housengw requested a review from lhstrh May 1, 2022 01:10
@lhstrh lhstrh merged commit 720d3b3 into master May 1, 2022
@lhstrh lhstrh deleted the prefix-print branch May 1, 2022 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c Related to C target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User facing utility functions such as info_print should also have lf_ prefix
2 participants