Simplify the mparse_open() interface.

Just return the file descriptor or -1 on error;
there is just one kind of error anyway.
Suggested by Christos Zoulas (NetBSD).

1 files changed, 1 insertions, 2 deletions

diff --git a/mandocdb.c b/mandocdb.c
index 3b089194..f222a752 100644
--- a/mandocdb.c
+++ b/mandocdb.c
@@ -1134,8 +1134,7 @@ mpages_merge(struct mparse *mp)
 		man = NULL;
 		sodest = NULL;
 
-		mparse_open(mp, &fd, mlink->file);
-		if (fd == -1) {
+		if ((fd = mparse_open(mp, mlink->file)) == -1) {
 			say(mlink->file, "&open");
 			goto nextpage;
 		}