todo about warning