Skip to content

fix: Pass useParentHandlers argument in LoggerImpl.setUseParentHandlers#157

Merged
bgrozev merged 1 commit intomasterfrom
claude/lucid-colden-bf58fa
Apr 21, 2026
Merged

fix: Pass useParentHandlers argument in LoggerImpl.setUseParentHandlers#157
bgrozev merged 1 commit intomasterfrom
claude/lucid-colden-bf58fa

Conversation

@bgrozev
Copy link
Copy Markdown
Member

@bgrozev bgrozev commented Apr 21, 2026

LoggerImpl.setUseParentHandlers ignored its argument and always passed false to the delegate, silently breaking any caller that passed true.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.01%. Comparing base (3c557c2) to head (aa74f72).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #157   +/-   ##
=========================================
  Coverage     42.01%   42.01%           
  Complexity      278      278           
=========================================
  Files            60       60           
  Lines          2449     2449           
  Branches        351      351           
=========================================
  Hits           1029     1029           
  Misses         1340     1340           
  Partials         80       80           
Files with missing lines Coverage Δ
...main/java/org/jitsi/utils/logging2/LoggerImpl.java 54.36% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c557c2...aa74f72. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bgrozev bgrozev merged commit ab4f756 into master Apr 21, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants