From 7fbec1d96a77b63a78b9e5f92d78648cdc60f220 Mon Sep 17 00:00:00 2001 From: Marek Kubica Date: Wed, 9 Mar 2022 17:57:55 +0100 Subject: [PATCH] Remove `validate_json` (#137) It doesn't do much and made only sense in atdgen when we used to have a `json` type. Now that the type doesn't exist it would require to be renamed into `validate_t` but such functionality doesn't seem to make sense to exist in Yojson, especially as it never does any validation to begin with. --- CHANGES.md | 2 ++ lib/read.mli | 7 ------- lib/read.mll | 2 -- 3 files changed, 2 insertions(+), 9 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index c435acec..c1779f2c 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -16,6 +16,8 @@ - Removed `json_max` type (@Leonidas-from-XIV, #103) - Removed constraint that the "root" value being rendered (via either `pretty_print` or `to_string`) must be an object or array. (@cemerick, #121) +- Removed `validate_json` as it only made sense if the type was called `json`. + (@Leonidas-from-XIV, #137) ### Add diff --git a/lib/read.mli b/lib/read.mli index 4d17e41c..7fca7cd4 100644 --- a/lib/read.mli +++ b/lib/read.mli @@ -261,12 +261,5 @@ val read_json : lexer_state -> Lexing.lexbuf -> t val skip_json : lexer_state -> Lexing.lexbuf -> unit val buffer_json : lexer_state -> Lexing.lexbuf -> unit -val validate_json : 'path -> t -> 'error option - (* always returns [None]. - Provided so that atdgen users can write: - - type t = abstract - *) - (* end undocumented section *) (**/**) diff --git a/lib/read.mll b/lib/read.mll index f45292e0..9e3cb54f 100644 --- a/lib/read.mll +++ b/lib/read.mll @@ -1212,6 +1212,4 @@ and junk = parse let compact ?std s = to_string (from_string s) - - let validate_json _path _value = None }