... that the Lazarus compiler isn't complaining about it. I'm finding your indenting style: begin if check_valid_switch_selected=False then EXIT; // no switch or no data for it. if switch_info_showing=True then begin show_info_button.Caption := 'show switch info'; resize_small(); switch_info_showing := False; end else begin show_switch_info(full_size_mm_radiobutton.Checked, switch_info_showing); show_info_button.Caption := 'hide switch info'; resize_big(); switch_info_showing := True; end; // ??? Is this next line required? ??? if switch_selector_listbox.Visible then switch_selector_listbox.SetFocus; end; very difficult to follow. I realise I will need to be flexible and less set in my ways for a collaborative project, but I'm wondering if there is a reason for not following my style: begin if check_valid_switch_selected=False then EXIT; // no switch or no data for it. if switch_info_showing=True then begin data_child_form.Close; show_info_button.Caption:= 'show switch &info'; switch_info_showing:=False; end else begin show_switch_info(full_size_mm_radiobutton.Checked,switch_info_showing); show_info_button.Caption:= 'hide ...