[nvda-addons] Tone master 1.1

  • From: "Joseph Lee" <joseph.lee22590@xxxxxxxxx>
  • To: <nvda-addons@xxxxxxxxxxxxx>
  • Date: Tue, 19 Jul 2016 14:12:20 -0700

Hi,

 

Review results:

 

*         Licensing: pass (carried from previous version)

*         Security: pass (thanks for addressing some issues).

*         User experience: pass with caution.

The way it was done was to review the diff between 1.0 and 1.1.

 

Comments:

 

*         Possible bug found: if you create an empty tone data file, select
it, attempt to play it, then select another one, NVDA will not play the
newly chosen tone data (will file a bug report on GitHub).

 

I recommend a green light once major bugs are resolved. Thanks. Reviews form
others are appreciated.

 

Cheers,

Joseph

Other related posts: