note on substring