From 377ca5c0733ec22fdf4998be04441849502dcf0d Mon Sep 17 00:00:00 2001
From: Gavin King <gavin@hibernate.org>
Date: Fri, 4 Sep 2020 08:02:49 +0200
Subject: [PATCH] use proper bullet list in Dialect javadoc

This is Javadoc, not a comment, so I should have
formatted it properly when I originally wrote this.
---
 .../java/org/hibernate/dialect/Dialect.java   | 150 +++++++++++-------
 1 file changed, 89 insertions(+), 61 deletions(-)

diff --git a/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java b/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java
index c1f7ba18f4..f967c14d9c 100644
--- a/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java
+++ b/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java
@@ -270,89 +270,117 @@ public abstract class Dialect implements ConversionContext {
 	 * These required functions include the functions defined by the JPA
 	 * query language specification:
 	 *
-	 *   * avg(arg)							- aggregate function
-	 *   * count([distinct ]arg)			- aggregate function
-	 *   * max(arg)							- aggregate function
-	 *   * min(arg)							- aggregate function
-	 *   * sum(arg)							- aggregate function
+	 * <ul>
+	 * <li> avg(arg)							- aggregate function
+	 * <li> count([distinct ]arg)				- aggregate function
+	 * <li> max(arg)							- aggregate function
+	 * <li> min(arg)							- aggregate function
+	 * <li> sum(arg)							- aggregate function
+	 * </ul>
 	 *
-	 *   * coalesce(arg0, arg1, ...)
-	 *   * nullif(arg0, arg1)
+	 * <ul>
+	 * <li> coalesce(arg0, arg1, ...)
+	 * <li> nullif(arg0, arg1)
+	 * </ul>
 	 *
-	 *   * lower(arg)
-	 *   * upper(arg)
-	 *   * length(arg)
-	 *   * concat(arg0, arg1, ...)
-	 *   * locate(pattern, string[, start])
-	 *   * substring(string, start[, length])
-	 *   * trim([[spec ][character ]from] string)
+	 * <ul>
+	 * <li> lower(arg)
+	 * <li> upper(arg)
+	 * <li> length(arg)
+	 * <li> concat(arg0, arg1, ...)
+	 * <li> locate(pattern, string[, start])
+	 * <li> substring(string, start[, length])
+	 * <li> trim([[spec ][character ]from] string)
+	 * </ul>
 	 *
-	 *   * abs(arg)
-	 *   * mod(arg0, arg1)
-	 *   * sqrt(arg)
+	 * <ul>
+	 * <li> abs(arg)
+	 * <li> mod(arg0, arg1)
+	 * <li> sqrt(arg)
+	 * </ul>
 	 *
-	 *   * current date
-	 *   * current time
-	 *   * current timestamp
+	 * <ul>
+	 * <li> current date
+	 * <li> current time
+	 * <li> current timestamp
+	 * </ul>
 	 *
 	 * Along with an additional set of functions defined by ANSI SQL:
 	 *
-	 *   * any(arg)							- aggregate function
-	 *   * every(arg)						- aggregate function
+	 * <ul>
+	 * <li> any(arg)							- aggregate function
+	 * <li> every(arg)							- aggregate function
+	 * </ul>
 	 *
-	 *   * cast(arg as Type)
-	 *   * extract(field from arg)
+	 * <ul>
+	 * <li> cast(arg as Type)
+	 * <li> extract(field from arg)
+	 * </ul>
 	 *
-	 *   * ln(arg)
-	 *   * exp(arg)
-	 *   * power(arg0, arg1)
-	 *   * floor(arg)
-	 *   * ceiling(arg)
+	 * <ul>
+	 * <li> ln(arg)
+	 * <li> exp(arg)
+	 * <li> power(arg0, arg1)
+	 * <li> floor(arg)
+	 * <li> ceiling(arg)
+	 * </ul>
 	 *
-	 *   * position(pattern in string)
-	 *   * substring(string from start[ for length])
-	 *   * overlay(string placing replacement from start[ for length])
+	 * <ul>
+	 * <li> position(pattern in string)
+	 * <li> substring(string from start[ for length])
+	 * <li> overlay(string placing replacement from start[ for length])
+	 * </ul>
 	 *
 	 * And the following functions for working with java.time types:
 	 *
-	 *   * local date
-	 *   * local time
-	 *   * local datetime
-	 *   * offset datetime
-	 *   * instant
+	 * <ul>
+	 * <li> local date
+	 * <li> local time
+	 * <li> local datetime
+	 * <li> offset datetime
+	 * <li> instant
+	 * </ul>
 	 *
 	 * And a number of additional "standard" functions:
 	 *
-	 *   * left(string, length)
-	 *   * right(string, length)
-	 *   * replace(string, pattern, replacement)
-	 *   * pad(string with length spec[ character])
+	 * <ul>
+	 * <li> left(string, length)
+	 * <li> right(string, length)
+	 * <li> replace(string, pattern, replacement)
+	 * <li> pad(string with length spec[ character])
+	 * </ul>
 	 *
-	 *   * sign(arg)
-	 *   * sin(arg)
-	 *   * cos(arg)
-	 *   * tan(arg)
-	 *   * asin(arg)
-	 *   * acos(arg)
-	 *   * atan(arg)
-	 *   * atan2(arg0, arg1)
-	 *   * round(arg0, arg1)
-	 *   * least(arg0, arg1, ...)
-	 *   * greatest(arg0, arg1, ...)
+	 * <ul>
+	 * <li> sign(arg)
+	 * <li> sin(arg)
+	 * <li> cos(arg)
+	 * <li> tan(arg)
+	 * <li> asin(arg)
+	 * <li> acos(arg)
+	 * <li> atan(arg)
+	 * <li> atan2(arg0, arg1)
+	 * <li> round(arg0, arg1)
+	 * <li> least(arg0, arg1, ...)
+	 * <li> greatest(arg0, arg1, ...)
+	 * </ul>
 	 *
-	 *   * format(datetime as pattern)
-	 *   * str(arg)					- synonym of cast(a as String)
-	 *   * ifnull(arg0, arg1)		- synonym of coalesce(a, b)
+	 * <ul>
+	 * <li> format(datetime as pattern)
+	 * <li> str(arg)					- synonym of cast(a as String)
+	 * <li> ifnull(arg0, arg1)			- synonym of coalesce(a, b)
+	 * </ul>
 	 *
 	 * Finally, the following functions are defined as abbreviations
 	 * for extract(), and desugared by the parser:
 	 *
-	 *   * second(arg)				- synonym of extract(second from a)
-	 *   * minute(arg)				- synonym of extract(minute from a)
-	 *   * hour(arg)				- synonym of extract(hour from a)
-	 *   * day(arg)					- synonym of extract(day from a)
-	 *   * month(arg)				- synonym of extract(month from a)
-	 *   * year(arg)				- synonym of extract(year from a)
+	 * <ul>
+	 * <li> second(arg)					- synonym of extract(second from a)
+	 * <li> minute(arg)					- synonym of extract(minute from a)
+	 * <li> hour(arg)					- synonym of extract(hour from a)
+	 * <li> day(arg)					- synonym of extract(day from a)
+	 * <li> month(arg)					- synonym of extract(month from a)
+	 * <li> year(arg)					- synonym of extract(year from a)
+	 * </ul>
 	 *
 	 */
 	public void initializeFunctionRegistry(QueryEngine queryEngine) {