goto ERROR するために status を↑で初期化してたり↓で
`status = r.status` と代入していたりで無駄な感覚。

ここは

```c
if( e != YRMCDS_OK ) {
    on_broken_connection_detected(c, e, YRMCDS_STATUS_OK TRSMLS_CC);
    return;
}
```

として `status` 変数と goto を削除してはどうだろう。