JakeWharton / JakeWharton/timber

Lint does not warn if format specifier missing

Open
#464 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Kotlin
Stars
10.8k
Forks
994
Avg merge
17h 16m
Merged PRs (30d)
17

Description

I added a debug statement
Timber.v("perm % is a valid combination %s",possibleCom,validCom);
instead of
Timber.v("perm %s is a valid combination %s",possibleCom,validCom);

There was no warning or error. The above bug (missing s after %) causes weird behavior.
In my case the subsequent library call post the debug log never completed.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing how Timber.v handles and validates format strings, using the two examples in the issue as the reproduction. Determine where a missing format specifier should be reported, then add regression coverage showing that the malformed call produces a warning or error while the corrected call remains valid.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, kotlin
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.