Skip to content

Remove unused lastCode property from the LZWStream class (PR 324 follow-up)#20733

Merged
Snuffleupagus merged 1 commit intomozilla:masterfrom
Snuffleupagus:LZWStream-rm-lastCode
Feb 25, 2026
Merged

Remove unused lastCode property from the LZWStream class (PR 324 follow-up)#20733
Snuffleupagus merged 1 commit intomozilla:masterfrom
Snuffleupagus:LZWStream-rm-lastCode

Conversation

@Snuffleupagus
Copy link
Collaborator

This appear to have been unused already in PR #324 all the way back in 2011.

…follow-up)

This appear to have been unused already in PR 324 all the way back in 2011.
@Snuffleupagus
Copy link
Collaborator Author

/botio-linux test

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/0e4a7c6c3c98234/output.txt

Copy link
Contributor

@calixteman calixteman left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you.

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Failed

Full output at http://54.241.84.105:8877/0e4a7c6c3c98234/output.txt

Total script time: 45.00 mins

  • Unit tests: FAILED
  • Integration Tests: Passed
  • Regression tests: FAILED
  different ref/snapshot: 1

Image differences available at: http://54.241.84.105:8877/0e4a7c6c3c98234/reftest-analyzer.html#web=eq.log

@Snuffleupagus Snuffleupagus merged commit 3c43414 into mozilla:master Feb 25, 2026
12 checks passed
@Snuffleupagus Snuffleupagus deleted the LZWStream-rm-lastCode branch February 25, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants