Skip to content

Un avertissement de compilation traîne depuis la #54 (Period.set déprécié) #62

Description

@InstaZDLL

Le défaut

Le dépôt tient une propriété : le build ne produit aucun avertissement de
compilation
. Elle est rompue depuis la PR #54.

w: app/src/test/java/app/waveflow/playback/TranscodeSeekingPlayerTest.kt:554:23
   'fun set(p0: Any?, p1: Any?, p2: Int, p3: Long, p4: Long): Timeline.Period'
   is deprecated. Deprecated in Java.

La fausse timeline de TranscodeSeekingPlayerTest appelle la surcharge de
Timeline.Period.set à cinq arguments, dépréciée dans media3 1.11 au profit de
celle qui prend isPlaceholder.

Comment il a été trouvé

En vérifiant la montée de version Dependabot (#56). L'avertissement n'est pas
dû au bump
: il apparaît à l'identique sur main (561a1d3), media3 n'y
bougeant pas. Vérifié en recompilant main dans une worktree séparée avec
--rerun-tasks.

Il n'a pas été vu au moment de la #54 parce qu'un build incrémental ne recompile
pas ce qu'il croit à jour : l'avertissement ne réapparaît qu'avec
--rerun-tasks ou après un clean
. C'est le vrai enseignement de ce ticket —
la propriété « zéro avertissement » ne se vérifie pas sur un build tiède.

Le correctif

Passer à la surcharge non dépréciée. C'est une fausse timeline de test : il n'y
a pas de comportement à préserver au-delà de ce que les tests existants
décrivent déjà, et aucun test de régression à écrire — il n'y avait pas de
défaut à faire réapparaître.

Ce qui éviterait la récidive

Rien dans la CI ne fait échouer le build sur un avertissement. Tant que ce n'est
pas le cas, la propriété repose sur la vigilance, et celle-ci a déjà cédé une
fois. À considérer : allWarningsAsErrors sur les tâches de compilation Kotlin.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions