There is one kind of combo box that populates automatically when you set 2 properties. There is a data bound combo box and list box. It has a rowsource property where you set the source of your data (recordsetname) and a listfield where you set the name of the field that the box is supposed to list.

You may have to switch the type of box combo box your using but the effort will pay for itself. Populating a combo box via code can be VERY time consuming if you have more than a couple of records. The rowsource and listfield properties can populate LOTS of records without slowing your application.

(I don't know if it will apply to your app but I usually set the style of the combo box to 2 and the matchentry to 1. That makes it really easy for the user to select data from the box.)