handzlikchris / handzlikchris/FastScriptReload

Generic class parameter isn't updated to patched version

Ouverte
#159 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
C#
Étoiles
2.2k
Forks
167
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

This

public class ZoomEvent : MouseEventBase<ZoomEvent> {
  public Vector2 Delta { get; private set; }

  public static ZoomEvent GetPooled(Vector2 delta) {
    var pooled = MouseEventBase<ZoomEvent>.GetPooled();
    pooled.Delta = delta * WheelEvent.scrollDeltaPerTick;
    // ...
    return pooled;
  }
}

produces following

public class ZoomEvent__Patched_: MouseEventBase<ZoomEvent> { // I guess there should be MouseEventBase<ZoomEvent__Patched_>?
  public Vector2 Delta { get; private set; }

  public static ZoomEvent__Patched_ GetPooled(Vector2 delta) {
    var pooled = MouseEventBase<ZoomEvent>.GetPooled(); // And here too?
    pooled.Delta = delta * WheelEvent.scrollDeltaPerTick;
    // ...
    return pooled;
  }
}

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser le code qui réécrit les noms et les références de types génériques lors de la production de classes C# patchées. Reproduisez l’exemple ZoomEvent et MouseEventBase de l’issue, puis inspectez la classe générée et l’appel à GetPooled. C’est terminé lorsque les paramètres de type patchés et les références associées utilisent systématiquement le nom de la classe patchée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
csharp, unity
Domaine
devtools, game-dev
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
32/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.