Except for missing brackets here, everything looks good