icsharpcode/ILSpy

Pointer arithmetic compound assignments

Open

#947 ouverte le 30 oct. 2017

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C# (3 645 forks)batch import
C#DecompilerEnhancementHelp Wanted

Métriques du dépôt

Stars
 (25 162 stars)
Métriques de merge PR
 (Merge moyen 15j 22h) (61 PRs mergées en 30 j)

Description

We don't support code like this:

someStruct.PointerField += i;
int* x = someStruct.PointerField++;

I'm not sure if it's worth adding support for these constructs -- these are two quite complex features interacting with each other; and such code is quite rare.

Guide contributeur

Direction de recherche
Étudiez la base de code d'ILSpy pour localiser où les expressions d'affectation composée sont traitées. Examinez la prise en charge existante pour l'arithmétique des pointeurs (par exemple, PointerArithmeticInstruction ou similaire) et voyez comment elle gère l'arithmétique simple des pointeurs sans affectation. Ensuite, examinez l'arbre d'expressions ou les nœuds AST pour les affectations composées (par exemple, CompoundAssignmentExpression) pour comprendre comment les étendre pour prendre en charge les cibles de pointeur. Les fichiers probablement impliqués incluent ICSharpCode.Decompiler/IL/ILInstruction.cs et les visiteurs associés. Vérifiez les tests existants pour l'arithmétique des pointeurs pour garantir qu'il n'y a pas de régressions.
Stack technique
csharp
Domaine
backendtooling
Type d'issue
Fonctionnalité
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
C#pointer arithmetic
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40