radareorg/radare2

Merge/Kill quintuplicate JSON implementations

Open

#12 870 ouverte le 21 janv. 2019

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C (3 229 forks)batch import
Hacktoberfestenhancementgood first issuejsonrefactor

Métriques du dépôt

Stars
 (23 826 stars)
Métriques de merge PR
 (Merge moyen 16h 24m) (99 PRs mergées en 30 j)

Description

Now we have:

  • pj.c
  • sdbjson
  • dsojson
  • handmade printf json
  • r_json

We must only keep pj.c and likely have sdb uses pj.c too.

Guide contributeur

Direction de recherche
Identifiez tous les fichiers d'implémentation JSON dans la base de code, analysez leurs interfaces et dépendances, et proposez un plan pour les consolider dans pj.c tout en veillant à ce que sdb utilise l'implémentation unifiée.
Stack technique
c
Domaine
backendtooling
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CCodebase navigation
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60