commiting fixed homework