changing to return void