Change sync fs calls to use assert style return values and errors

Start to #52
we don't have a hard and fast rule, but we sorta go off this one:

```
http://httpd.apache.org/dev/styleguide.html
```