Skip to content

Bugfix/equal null object#1351

Open
darkdread wants to merge 2 commits intoremotestorage:masterfrom
darkdread:bugfix/equal-null-object
Open

Bugfix/equal null object#1351
darkdread wants to merge 2 commits intoremotestorage:masterfrom
darkdread:bugfix/equal-null-object

Conversation

@darkdread
Copy link

Fix equality checking for null type

Walson Low (刘维昇) added 2 commits February 19, 2026 15:33
Updated build files to sync with code (desync since
remotestorage@8a69627)
Without this fix, the equal function could potentially throw a TypeError
due to the variable holding a value of null.
@darkdread darkdread force-pushed the bugfix/equal-null-object branch from 22d7dc3 to 206ca97 Compare February 19, 2026 07:34
Copy link
Member

@raucao raucao left a comment

Choose a reason for hiding this comment

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

LGTM. 👍 And many thanks for porting one of the remaining test suites in the process!

Could you perhaps remove the release directory changes from the branch/PR so that they get built only by the release/version script?

It could also be useful to add a short description of how you actually encountered this bug, so we can document the fix properly in the release notes.

Copy link
Contributor

@DougReeder DougReeder left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments