abseil / abseil/abseil-py

log_first_n implementation doesn't match doc, ignores formatting args

Offen
#142 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
2.5k
Forks
279
Ø Merge
2 T. 1 Std.
Gemergte PRs (30 T.)
1

Beschreibung

The log_first_n doc says:

`Logs 'msg % args' at level 'level' only first 'n' times`

However, the implementation only tracks the caller's file and line number, and ignores the formatting args. So if I do something like:

```
for arg in ['arg1', 'arg2']:
log_first_n('%s', 1, arg)
```

It will never log for arg2.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.