Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
The algorithm was already based on bytes internally. Also use byte literals instead of casting u8 to char for matching.
- Loading branch information
1 parent
e0a6e2b
commit 56218f5
Showing
1 changed file
with
29 additions
and
17 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
56218f5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from alexcrichton
at SimonSapin@56218f5
56218f5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging SimonSapin/rust/base64-from-bytes = 56218f5 into auto
56218f5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SimonSapin/rust/base64-from-bytes = 56218f5 merged ok, testing candidate = f15d6d2
56218f5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/434
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/434
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/433
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/434
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/436
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/437
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/436
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/442
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/437
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/436
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/439
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/437
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/437
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/438
56218f5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = f15d6d2