[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#63985] [PATCH v4 1/5] services: configuration: Simplify normalize-e
From: |
Bruno Victal |
Subject: |
[bug#63985] [PATCH v4 1/5] services: configuration: Simplify normalize-extra-args. |
Date: |
Sat, 7 Oct 2023 16:59:06 +0100 |
* gnu/services/configuration.scm
(define-configuration-helper, normalize-extra-args): Use #f instead of
%unset-value.
---
gnu/services/configuration.scm | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/gnu/services/configuration.scm b/gnu/services/configuration.scm
index 367b85c1be..ecc5049a79 100644
--- a/gnu/services/configuration.scm
+++ b/gnu/services/configuration.scm
@@ -190,32 +190,32 @@ (define (define-configuration-helper serialize?
serializer-prefix syn)
(define (normalize-extra-args s)
"Extract and normalize arguments following @var{doc}."
(let loop ((s s)
- (sanitizer* %unset-value)
- (serializer* %unset-value))
+ (sanitizer* #f)
+ (serializer* #f))
(syntax-case s (sanitizer serializer empty-serializer)
(((sanitizer proc) tail ...)
- (if (maybe-value-set? sanitizer*)
- (syntax-violation 'sanitizer "duplicate entry"
- #'proc)
+ (if sanitizer*
+ (syntax-violation 'sanitizer
+ "duplicate entry" #'proc)
(loop #'(tail ...) #'proc serializer*)))
(((serializer proc) tail ...)
- (if (maybe-value-set? serializer*)
- (syntax-violation 'serializer "duplicate or conflicting entry"
- #'proc)
+ (if serializer*
+ (syntax-violation 'serializer
+ "duplicate or conflicting entry" #'proc)
(loop #'(tail ...) sanitizer* #'proc)))
((empty-serializer tail ...)
- (if (maybe-value-set? serializer*)
+ (if serializer*
(syntax-violation 'empty-serializer
"duplicate or conflicting entry" #f)
(loop #'(tail ...) sanitizer* #'empty-serializer)))
(() ; stop condition
(values (list sanitizer* serializer*)))
((proc) ; TODO: deprecated, to be removed.
- (null? (filter-map maybe-value-set? (list sanitizer* serializer*)))
+ (not (or sanitizer* serializer*))
(begin
(warning #f (G_ "specifying serializers after documentation is \
deprecated, use (serializer ~a) instead~%") (syntax->datum #'proc))
- (values (list %unset-value #'proc)))))))
+ (values (list #f #'proc)))))))
(syntax-case syn ()
((_ stem (field field-type+def doc extra-args ...) ...)
@@ -239,11 +239,11 @@ (define (define-configuration-helper serialize?
serializer-prefix syn)
default-value))
#'((field-type def) ...)))
((field-sanitizer ...)
- (map maybe-value #'(sanitizer* ...)))
+ #'(sanitizer* ...))
((field-serializer ...)
(map (lambda (type proc)
(and serialize?
- (or (maybe-value proc)
+ (or proc
(if serializer-prefix
(id #'stem serializer-prefix #'serialize-
type)
(id #'stem #'serialize- type)))))
--
2.41.0
- [bug#63985] [PATCH v4 0/5] SRFI-171 based improvements for define-configuration, Bruno Victal, 2023/10/07
- [bug#63985] [PATCH v4 2/5] services: configuration: Use transducers within serialize-configuration., Bruno Victal, 2023/10/07
- [bug#63985] [PATCH v4 5/5] services: configuration: Add some commonly used predicates., Bruno Victal, 2023/10/07
- [bug#63985] [PATCH v4 4/5] doc: Rewrite define-configuration., Bruno Victal, 2023/10/07
- [bug#63985] [PATCH v4 3/5] services: fstrim-service-type: Serialize with SRFI-171 transducers., Bruno Victal, 2023/10/07
- [bug#63985] [PATCH v4 1/5] services: configuration: Simplify normalize-extra-args.,
Bruno Victal <=
- bug#63985: SRFI-171 based improvements for define-configuration, Maxim Cournoyer, 2023/10/07