Skip to content

Update the Lilli braille display driver to be 64-bit#19131

Merged
SaschaCowley merged 1 commit into
masterfrom
lilli64
Oct 24, 2025
Merged

Update the Lilli braille display driver to be 64-bit#19131
SaschaCowley merged 1 commit into
masterfrom
lilli64

Conversation

@SaschaCowley

Copy link
Copy Markdown
Member

Link to issue number:

None

Summary of the issue:

Lilli.dll is a 32-bit DLL, so the MDV Lilli display can't be used with 64-bit builds of NVDA.

Description of user facing changes:

TheLilli display can be used again.

Description of developer facing changes:

None

Description of development approach:

Update miscDeps

Testing strategy:

This DLL was reported to work by MDV.

Known issues with pull request:

None

Code Review Checklist:

  • Documentation:
    • Change log entry
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

Copilot AI review requested due to automatic review settings October 23, 2025 03:51
@SaschaCowley SaschaCowley requested a review from a team as a code owner October 23, 2025 03:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the MDV Lilli braille display driver to support 64-bit NVDA builds by updating the miscDeps submodule, which contains a new 64-bit compatible version of the Lilli.dll library.

Key Changes:

  • Updated miscDeps submodule to include 64-bit Lilli.dll

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gerald-hartig

Copy link
Copy Markdown
Contributor

What's the origin of the DLL? Who provided it to us / where did we get it from?

@SaschaCowley

Copy link
Copy Markdown
Member Author

@gerald-hartig

What's the origin of the DLL? Who provided it to us / where did we get it from?

The DLL was provided to us by MDV via email.

@SaschaCowley SaschaCowley merged commit 9e37e57 into master Oct 24, 2025
121 of 133 checks passed
@SaschaCowley SaschaCowley deleted the lilli64 branch October 24, 2025 04:36
@github-actions github-actions Bot added this to the 2026.1 milestone Oct 24, 2025
bramd pushed a commit to bramd/nvda that referenced this pull request Nov 11, 2025
## Summary of the issue:

Lilli.dll is a 32-bit DLL, so the MDV Lilli display can't be used with
64-bit builds of NVDA.

### Description of user facing changes:

TheLilli display can be used again.

### Description of developer facing changes:

None

### Description of development approach:

Update miscDeps

### Testing strategy:

This DLL was reported to work by MDV.

### Known issues with pull request:

None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants