search

Ayuget / Redface

Android client for HFR's forum
Apache License 2.0
42 stars 21 forks source link

Liste noire. #180

Closed nbonnec closed 7 years ago

nbonnec commented 7 years ago

Corrige aussi #158

Ayuget commented 7 years ago

Un grand merci pour ta contribution, je merge et je vais tester la fonctionnalité :)

Je viens de relire le code ca m'a l'air OK.

Je me faisait quand même la réflexion qu'il faudrait que je profile le code de parsing des pages, car vu que le parsing se fait à la bourrin via des regex (pour les fonctionnalités comme EgoQuote ou la Blacklist), j'ai peur qu'à la longue on pourrisse le temps de chargement.

Mais si ca trouve je me fais des films, l'idéal serait de quantifier tout ça.

nbonnec commented 7 years ago

Oui j'y ai pensé aussi. J'ai fait des tests rapidement en blindant la BL et ça n'a pas l'air d'avoir d'incidence. Mais mesurer tout ça ça serait pas mal.

Le 23 janv. 2017 10:32 PM, "Ayuget" notifications@github.com a écrit :

Un grand merci pour ta contribution, je merge et je vais tester la fonctionnalité :)

Je viens de relire le code ca m'a l'air OK.

Je me faisait quand même la réflexion qu'il faudrait que je profile le code de parsing des pages, car vu que le parsing se fait à la bourrin via des regex (pour les fonctionnalités comme EgoQuote ou la Blacklist), j'ai peur qu'à la longue on pourrisse le temps de chargement.

Mais si ca trouve je me fais des films, l'idéal serait de quantifier tout ça.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Ayuget/Redface/pull/180#issuecomment-274623789, or mute the thread https://github.com/notifications/unsubscribe-auth/AAcPAVuITmHYSFGrF4RqShZE4b99JEiHks5rVRxxgaJpZM4LJ7Hh .