my first checkin on a branch

are you guys agree with this changes?