added TODO about allocating memory or not