Skip to content

fix: Mark get_log_level() pure for consistency (fixes #1347) #1348

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Sep 19, 2025
Merged

fix: Mark get_log_level() pure for consistency (fixes #1347) #1348

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b2af879
fix: mark get_log_level() pure for consistency (fixes #1347)
krystophny Sep 19, 2025
00656d2
style: wrap long lines in fortplot_logging.f90 to comply with 88–90 c…
krystophny Sep 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions src/external/fortplot_logging.f90
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ module fortplot_logging
private

public :: set_log_level, get_log_level, log_info, log_warning, log_error, log_debug
public :: LOG_LEVEL_SILENT, LOG_LEVEL_ERROR, LOG_LEVEL_WARNING, LOG_LEVEL_INFO, LOG_LEVEL_DEBUG
public :: LOG_LEVEL_SILENT, LOG_LEVEL_ERROR, &
LOG_LEVEL_WARNING, LOG_LEVEL_INFO, LOG_LEVEL_DEBUG
public :: initialize_warning_suppression, is_warnings_suppressed

! Log levels (in increasing verbosity)
Expand Down Expand Up @@ -47,7 +48,7 @@ subroutine set_log_level(level)
current_log_level = lvl
end subroutine set_log_level

function get_log_level() result(level)
pure function get_log_level() result(level)
!! Get the current global logging level
!! Returns one of: LOG_LEVEL_SILENT, LOG_LEVEL_ERROR,
!! LOG_LEVEL_WARNING, LOG_LEVEL_INFO, LOG_LEVEL_DEBUG
Expand Down Expand Up @@ -120,7 +121,8 @@ subroutine initialize_warning_suppression()
end if

! Check for manual warning suppression override
call get_environment_variable('FORTPLOT_SUPPRESS_WARNINGS', env_value, status=status)
call get_environment_variable('FORTPLOT_SUPPRESS_WARNINGS', env_value, &
status=status)
if (status == 0 .and. len_trim(env_value) > 0) then
warnings_suppressed = parse_boolean_env(env_value)
suppression_initialized = .true.
Expand Down
Loading