safe-global / safe-smart-account / 16190880859

94%

push

github

Refactor `domainSeparator` implementation (#1016)

We give it the same treatment as `getTransactionHash`, as it has
sufficiently substantial gains IMO, and the assembly is simple enough.

**Before:**

```
$ npm run codesize | grep '^Safe '
Safe 21434 bytes (limit is 24576)

$ npm run benchmark | grep -A4 'ERC20 - transfer' | sed '1P;$!N;$!D'
  ERC20 - transfer
           Used 82498n gas for >transfer<
    ✔ with a single owner Safe
```

**After:**

```
$ npm run codesize | grep '^Safe '
Safe 21372 bytes (limit is 24576)

$ npm run benchmark | grep -A4 'ERC20 - transfer' | sed '1P;$!N;$!D'
  ERC20 - transfer
           Used 82382n gas for >transfer<
    ✔ with a single owner Safe
```

308 of 340 branches covered (90.59%)

Branch coverage included in aggregate %.

450 of 462 relevant lines covered (97.4%)

109.05 hits per line