From a conceptual point of view, I understand this is a copy constructor that creates a ContextAttribute from another ContextAttributed passed as argument (I'm not completely sure, as the diff in github.com cuts the upper part).

In that case I think that a copy should be done... not only for the 6 invocations to the constructor that currently have (that maybe doesn't need an actual copy), but also to maintain a coherent semantics to avoid problems in potential future invocations to the constructor.