space around operators here too.  The parenthesis are redundant, so please remove them:

```
  uint32_t j = str->len - 1;
```