unecessary loop