get rid of regex_none()

- Remove `regex_none()`, document `NULL` default.

Author: krlmlr

API

@@ -3,13 +3,16 @@
 ## Exported functions
 
 create_style_guide(initialize = initialize_attributes, line_break = NULL, space = NULL, token = NULL, indention = NULL, use_raw_indention = FALSE, reindention = tidyverse_reindention())
-regex_none()
 specify_math_token_spacing(zero = NULL, one = c("'+'", "'-'", "'*'", "'/'", "'^'"))
-specify_reindention(regex_pattern = regex_none(), indention = 0, comments_only = TRUE)
+specify_reindention(regex_pattern = NULL, indention = 0, comments_only = TRUE)
 style_dir(path = ".", ..., style = tidyverse_style, transformers = style(...), filetype = "R", recursive = TRUE, exclude_files = NULL)
 style_file(path, ..., style = tidyverse_style, transformers = style(...))
 style_pkg(pkg = ".", ..., style = tidyverse_style, transformers = style(...), filetype = "R", exclude_files = "R/RcppExports.R")
 style_text(text, ..., style = tidyverse_style, transformers = style(...))
 tidyverse_math_token_spacing()
 tidyverse_reindention()
 tidyverse_style(scope = "tokens", strict = TRUE, indent_by = 2, start_comments_with_one_space = FALSE, reindention = tidyverse_reindention(), math_token_spacing = tidyverse_math_token_spacing())
+
+## Foreign S3 methods
+
+print.vertical(x, ...)

NAMESPACE

@@ -2,7 +2,6 @@
 
 S3method(print,vertical)
 export(create_style_guide)
-export(regex_none)
 export(specify_math_token_spacing)
 export(specify_reindention)
 export(style_dir)

R/reindent.R

@@ -154,15 +154,3 @@ set_regex_indention <- function(flattened_pd,
   bind_rows(to_check, not_to_check) %>%
     arrange(pos_id)
 }
-
-#' Return regex patterns for re-indention
-#'
-#' @name regex_for_reindention
-NULL
-
-#' @export
-#' @describeIn regex_for_reindention Returns `NULL`, i.e. no pattern will
-#'   match against this.
-regex_none <- function() {
-  NULL
-}

R/style_guides.R

@@ -194,7 +194,7 @@ create_style_guide <- function(initialize = initialize_attributes,
 #' defaults, so the user can specify deviations from them conveniently without
 #' the need of setting all arguments explicitly.
 #' @param regex_pattern Character vector with regular expression patterns that
-#'   are to be re-indented with spaces.
+#'   are to be re-indented with spaces, `NULL` if no reindention needed.
 #' @param indention The indention tokens should have if they match
 #'   `regex_pattern`.
 #' @param comments_only Whether the `regex_reindention_pattern` should only be
@@ -206,7 +206,7 @@ NULL
 #' @describeIn reindention Allows to specify which tokens are reindented and
 #'   how.
 #' @export
-specify_reindention <- function(regex_pattern = regex_none(),
+specify_reindention <- function(regex_pattern = NULL,
                                 indention = 0,
                                 comments_only = TRUE)
   lst(
@@ -221,7 +221,7 @@ specify_reindention <- function(regex_pattern = regex_none(),
 #' @export
 tidyverse_reindention <- function() {
   specify_reindention(
-    regex_pattern = regex_none(), indention = 0, comments_only = TRUE
+    regex_pattern = NULL, indention = 0, comments_only = TRUE
   )
 }
 

man/regex_for_reindention.Rd

@@ -13,7 +13,7 @@ test_that("do not force regex token-dependent indention without pattern", {
   expect_warning(test_collection(
     "unindention_regex", "regex_force_no",
     transformer = style_text,
-    reindention = specify_reindention(regex_none())), NA)
+    reindention = specify_reindention(NULL)), NA)
 })