Reconsider how the change type dialog handles newly entered types

オープン
#4,965 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

@CouleeApps がすでに取り組んでいます。

2024年4月4日 から。

評価

この issue はまだ評価されていません。

説明

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.

主要言語
C++
スター
1.3k
フォーク
298
平均マージ
5日 5時間
マージ済み PR（30日）
19

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Vector35/binaryninja-api のほかの issue

Vector35/binaryninja-api の issue をすべて見る

似ている issue

C++ の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。