JurkMonkey raised good points about event args. However, based on your code, which is likely to have been implemented on the client (eg. Form)
Code:
this.combobox_UserChanged(combobox, null)
I suppose you do not need the event argument. I was only saying it's okay to pass null because it will compile and run.
I did mention...
And, about the event args param, if the type is System.EventArgs, yes it's safe to pass null, (try also EventArgs.Empty).
Looking at .Net convention, most event handlers with System.EvenArgs type of event args are so because they do not need to pass any information back to the client, eg. Button.Click, Form.Load. But, events that do need to send back some pieces of information subclass System.EventArgs to strongly-typed ones and add the additional information, like MouseEventHandler for Control.MouseX events, DataRowChangedEventHandler for DataTable.RowChanged event.
(However, this may not be true if the control passes an strongly-typed/subclassed EventArgs despite the event arg being a System.EventArgs, aka polymorphism. Very inconvenient though).
Seeing that you used the System.EventArgs as the data type for UserChanged event arg, based on the above convention, and the control I assume is intended for your application only, I stated (my opinion only) that it's safe to pass null or System.EventArgs.Empty.
My 2cents. Hope this helps.
