Possible problem in org.apache.cxf.endpoint.ClientImpl.processResult()?

[Gary Gregory]

In master, at
org.apache.cxf.endpoint.ClientImpl.processResult(ClientImpl.java:663), we
throw a IllegalEmptyResponseException where I see:

            if (!boi.getOutput().getMessageParts().isEmpty()) {
                //we were supposed to get some output, but didn't.
                throw new IllegalEmptyResponseException("Response message
did not contain proper response data."
                    + " Expected: " +
boi.getOutput().getMessageParts().get(0).getConcreteName());
            }

The test does not seem right:

!boi.getOutput().getMessageParts().isEmpty()

Should the "!" really be there if we want an error is there is no response?

Gary