Perhaps instead of returning silently, this should throw an exception?