8000 Comparing v1.1.2...v1.1.3 · google/uuid · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: google/uuid
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.1.2
Choose a base ref
...
head repository: google/uuid
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.1.3
Choose a head ref
  • 1 commit
  • 2 files changed
  • 1 contributor

Commits on Dec 30, 2020

  1. Use a custom error type for invalid lengths, replacing fmt.Errorf (#69

    )
    
    * Add benchmarks for different kinds of invalid UUIDs
    
    Also add a test case for too-short UUIDs to ensure behavior doesn’t
    change.
    
    * Use a custom error type for invalid lengths, replacing `fmt.Errorf`
    
    This significantly improves the speed of failed parses due to wrong
    lengths. Previously the `fmt.Errorf` call dominated, making this the
    most expensive error and more expensive than successfully parsing:
    
        BenchmarkParse-4                 29226529        36.1 ns/op
        BenchmarkParseBadLength-4         6923106       174 ns/op
        BenchmarkParseLen32Truncated-4   26641954        38.1 ns/op
        BenchmarkParseLen36Corrupted-4   19405598        59.5 ns/op
    
    When the formatting is not required and done on-demand, the failure per
    se is much faster:
    
        BenchmarkParse-4                 29641700        36.3 ns/op
        BenchmarkParseBadLength-4        58602537        20.0 ns/op
        BenchmarkParseLen32Truncated-4   30664791        43.6 ns/op
        BenchmarkParseLen36Corrupted-4   18882410        61.9 ns/op
    joewreschnig authored Dec 30, 2020
    Configuration menu
    Copy the full SHA
    edef28d View commit details
    Browse the repository at this point in the history
Loading
0