This fn is called `all-walls`, but we pass it `walls`. Can we change one or both of these names to make it clearer what's going on?