carterkozak commented on pull request #237: URL: https://github.com/apache/httpcomponents-core/pull/237#issuecomment-744743319
A common prefix would be similar to adding these methods directly to `StatusClass`, for example ```java // original: statusLine.isError(); // prefix proposal // At a glance this looks like we're checking for malformed status lines statusLine.isStatusError(); // alternative StatusClass utility functionality statusLine.getStatusClass().isError(); ``` Usage wouldn't be quite as concise as the current proposal, but it might be a better fit than extending method names. That said, I don't have a preference one way or the other, just pointing out an alternative. The utility functionality will be helpful. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
