getting syntax correct etc