先に == 255 の判定をして 0 に戻すから
範囲外の buffer[255] にアクセスする危険があるのか。

インデクスに前置++は見慣れなくて気色悪い、じゃ済まない問題だな。