From de038270f72214b169d056642eb7144a79e6f126 Mon Sep 17 00:00:00 2001
From: Emilio Jesus Gallego Arias
Date: Thu, 7 Jul 2016 04:56:24 +0200
Subject: Unify location handling of error functions.

In some cases prior to this patch, there were two cases for the same
error function, one taking a location, the other not.

We unify them by using an option parameter, in the line with recent
changes in warnings and feedback.

This implies a bit of clean up in some places, but more importantly, is
the preparation for subsequent patches making `Loc.location` opaque,
change that could be use to improve modularity and allow a more
functional implementation strategy --- for example --- of the
beautifier.
---
 dev/doc/changes.txt | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'dev/doc/changes.txt')

diff --git a/dev/doc/changes.txt b/dev/doc/changes.txt
index fcee79e07e..ac874befd1 100644
--- a/dev/doc/changes.txt
+++ b/dev/doc/changes.txt
@@ -1,3 +1,13 @@
+=========================================
+= CHANGES BETWEEN COQ V8.6 AND COQ V8.7 =
+=========================================
+
+** Error handling **
+
+All error functions now take an optional parameter `?loc:Loc.t`. For
+functions that used to carry a suffix `_loc`, such suffix has been
+dropped.
+
 =========================================
 = CHANGES BETWEEN COQ V8.5 AND COQ V8.6 =
 =========================================
-- 
cgit v1.2.3


From 543ee0c7ad43874c577416af9f2e5a94d7d1e4d3 Mon Sep 17 00:00:00 2001
From: Emilio Jesus Gallego Arias
Date: Fri, 19 Aug 2016 01:58:04 +0200
Subject: Remove errorlabstrm in favor of user_err

As noted by @ppedrot, the first is redundant. The patch is basically a renaming.

We didn't make the component optional yet, but this could happen in a
future patch.
---
 dev/doc/changes.txt | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

(limited to 'dev/doc/changes.txt')

diff --git a/dev/doc/changes.txt b/dev/doc/changes.txt
index ac874befd1..699130944a 100644
--- a/dev/doc/changes.txt
+++ b/dev/doc/changes.txt
@@ -2,11 +2,15 @@
 = CHANGES BETWEEN COQ V8.6 AND COQ V8.7 =
 =========================================
 
+* ML API *
+
 ** Error handling **
 
-All error functions now take an optional parameter `?loc:Loc.t`. For
-functions that used to carry a suffix `_loc`, such suffix has been
-dropped.
+- All error functions now take an optional parameter `?loc:Loc.t`. For
+  functions that used to carry a suffix `_loc`, such suffix has been
+  dropped.
+
+- `errorlabstrm` has been removed in favor of `user_err`.
 
 =========================================
 = CHANGES BETWEEN COQ V8.5 AND COQ V8.6 =
-- 
cgit v1.2.3


From fc579fdc83b751a44a18d2373e86ab38806e7306 Mon Sep 17 00:00:00 2001
From: Emilio Jesus Gallego Arias
Date: Fri, 19 Aug 2016 02:35:47 +0200
Subject: Make the user_err header an optional parameter.

Suggested by @ppedrot
---
 dev/doc/changes.txt | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'dev/doc/changes.txt')

diff --git a/dev/doc/changes.txt b/dev/doc/changes.txt
index 699130944a..7d5e3d1fe4 100644
--- a/dev/doc/changes.txt
+++ b/dev/doc/changes.txt
@@ -12,6 +12,8 @@
 
 - `errorlabstrm` has been removed in favor of `user_err`.
 
+- The header parameter to `user_err` has been made optional.
+
 =========================================
 = CHANGES BETWEEN COQ V8.5 AND COQ V8.6 =
 =========================================
-- 
cgit v1.2.3