radareorg/radare2

Implement asm.tabs2

Open

#14.455 geöffnet am 29. Juni 2019

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C (3.229 Forks)batch import
FEEDBACK WANTEDRAsm-Disassemblerconsoleuigood first issue

Repository-Metriken

Stars
 (23.826 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 16h 24m) (99 gemergte PRs in 30 T)

Beschreibung

My idea is that we should have two asm.tabs variables (names can be improved, im open to proposals).

  • asm.tabs = specify how many spaces between opcode and 1st operand
  • asm.tabs2 = specify space between operands

this will be useful for architectures with long instruction names like dalvik

Contributor Guide

Research-Richtung
Das Issue fordert eine neue Konfigurationsvariable `asm.tabs2` an, um den Abstand zwischen Operanden in der Assembler Ausgabe zu steuern. Untersuche die vorhandene Implementierung von `asm.tabs` im radare2 Quellcode, wahrscheinlich in Dateien im Zusammenhang mit der Assembler Konfiguration oder dem asm Plugin. Das Ziel ist, eine neue Variable hinzuzufügen, sie zu parsen und die Logik für den Operandenabstand zu ändern. Prüfe die aktuelle Behandlung von `asm.tabs`, um zu verstehen, wie sie für `asm.tabs2` repliziert werden kann. Es gibt einen Zugewiesenen, aber keinen verlinkten PR, daher ist die Arbeit noch offen.
Tech Stack
c
Domain
clitooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
CGitradare2 build system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50