nim-works/nimskull

Clean up `ast.TIdTable` type definition

Aperta

#145 aperta il 31 dic 2021

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Nim (39 fork)auto 404
good first issuerefactor

Metriche repository

Star
 (346 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

ast defines TIdTable type that is used in several parts of the semantic pass. Type defined as follows:

  TIdPair* = object
    key*: PIdObj
    val*: RootRef

  TIdPairSeq* = seq[TIdPair]
  TIdTable* = object # the same as table[PIdent] of PObject
    counter*: int
    data*: TIdPairSeq

All the uses of the val: RootRef field are either converted to PType or PSym (result = PType(idTableGet(c.bindings, t)) etc.). This means RootRef can be easily replaced with

type
  PSymOrTyp = object
    case isSym*: bool
      of true:
        sym*: PSym

      of false:
        typ*: PType

This would not create any functional difference, but will clean up the code a little.

Guida contributor

Direzione di ricerca
Esamina l'utilizzo di TIdTable nel codebase, in particolare il campo val. Comprendi come vengono usati PType e PSym, quindi sostituisci RootRef con la variante oggetto proposta per tipizzare staticamente il campo val.
Tech stack
Nessuno
Dominio
compilers
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nim
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70