Reconsider how the change type dialog handles newly entered types

Aperta
#4,965 2 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@CouleeApps ci sta già lavorando.

Dal 4/4/2024.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

Component: UI Effort: Low Impact: High UI: Dialogs

Currently, when attempting to change a type for a variable or parameter, BN tests if the new type is correct as we type it, before clicking on the Accept button. However, this process can cause lags and make the 'Change Type' dialog unresponsive, especially if an incorrect type is entered.

You can reproduce this by

  1. Clicking on any variable or parameter and pressing Y
  2. Providing a random type, such as "asdasd123"
  3. Pressing Enter a few times (in order to make it more clear)
  4. See how the change type dialog becomes unresponsive

Here is an example GIF:

Recording

I encounter this issue in every RE session, and it often leads to wasted time. I suggest that BN performs type checking only when we press Enter or click on the 'Accept' button. This would eliminate the lag issue and allow for more appropriate error messages to be displayed regarding the error. In current implementation, we need to hover over the "Error parsing specified type" message to see details, which can be a bit difficult to notice.

Lingua principale
C++
Stelle
1.3k
Fork
298
Merge medio
5g 5h
PR unite (30g)
19

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Vector35/binaryninja-api

Tutte le issue di Vector35/binaryninja-api

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.