what is the purpose of unexported variable 'length'? Can't you just call len(Data) ?