Skip to content

Conversation

@jschwe
Copy link
Member

@jschwe jschwe commented Nov 20, 2025

Switches to log methods over eprintln in cases where it makes sense. stdout / stderr output may be difficult to view on devices like android / ohos, and has no filter options, so we should prefer log methods.
This PR avoids prints, which are controlled by a debugging option.

Testing: Only changes logging, no functional behavior change.

Switches to log methods over eprintln in cases where it makes sense.
stdout / stderr output may be difficult to view on devices like
android / ohos, and has no filter options, so we should prefer
log methods.
This PR avoids prints, which are controlled by a debugging option.

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 20, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 20, 2025
@jschwe jschwe added this pull request to the merge queue Nov 20, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 20, 2025
Merged via the queue into servo:main with commit f76ce4c Nov 20, 2025
36 checks passed
@jschwe jschwe deleted the jschwender/avoid_print branch November 20, 2025 15:56
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 20, 2025
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.

3 participants