Polishing

Author: jhoeller

spring-messaging/src/main/java/org/springframework/messaging/converter/ProtobufMessageConverter.java

@@ -48,15 +48,15 @@
  * <p>This converter supports by default {@code "application/x-protobuf"} with the official
  * {@code "com.google.protobuf:protobuf-java"} library.
  *
- * <p>{@code "application/json"} can be supported with the official {@code "com.google.protobuf:protobuf-java-util"} 3.x
- *  with 3.3 or higher recommended
+ * <p>{@code "application/json"} can be supported with the official
+ * {@code "com.google.protobuf:protobuf-java-util"} 3.x, with 3.3 or higher recommended.
  *
  * @author Parviz Rozikov
+ * @author Rossen Stoyanchev
  * @since 5.2.2
  */
 public class ProtobufMessageConverter extends AbstractMessageConverter {
 
-
 	/**
 	 * The default charset used by the converter.
 	 */
@@ -187,7 +187,7 @@ else if (this.protobufFormatSupport != null) {
 			}
 		}
 		catch (IOException ex) {
-			throw new MessageConversionException("Could not write proto message" + ex.getMessage(), ex);
+			throw new MessageConversionException("Failed to print Protobuf message: " + ex.getMessage(), ex);
 
 		}
 		return payload;

spring-web/src/main/java/org/springframework/http/converter/FormHttpMessageConverter.java

@@ -408,9 +408,8 @@ private void writeForm(MultiValueMap<String, Object> formData, @Nullable MediaTy
 	 * {@linkplain #setCharset(Charset) charset} if it does not have one.
 	 * If {@code contentType} is {@code null},
 	 * {@code application/x-www-form-urlencoded; charset=UTF-8} is returned.
-	 *
 	 * <p>Subclasses can override this method to change this behavior.
-	 * @param contentType the preferred content type, can be {@code null}
+	 * @param contentType the preferred content type (can be {@code null})
 	 * @return the content type to be used
 	 * @since 5.2.2
 	 */

spring-webflux/src/main/java/org/springframework/web/reactive/function/client/DefaultExchangeStrategiesBuilder.java

@@ -29,6 +29,7 @@
  * Default implementation of {@link ExchangeStrategies.Builder}.
  *
  * @author Arjen Poutsma
+ * @author Brian Clozel
  * @since 5.0
  */
 final class DefaultExchangeStrategiesBuilder implements ExchangeStrategies.Builder {
@@ -79,7 +80,6 @@ private static class DefaultExchangeStrategies implements ExchangeStrategies {
 
 		private final List<HttpMessageWriter<?>> writers;
 
-
 		public DefaultExchangeStrategies(ClientCodecConfigurer codecConfigurer) {
 			this.codecConfigurer = codecConfigurer;
 			this.readers = unmodifiableCopy(this.codecConfigurer.getReaders());
@@ -90,9 +90,8 @@ private static <T> List<T> unmodifiableCopy(List<? extends T> list) {
 			return Collections.unmodifiableList(new ArrayList<>(list));
 		}
 
-
-		@Override
 		@Deprecated
+		@Override
 		public Builder mutate() {
 			return new DefaultExchangeStrategiesBuilder(this);
 		}