Revert "emit a message instead of a warning using an internal wrapper"

This reverts commit 9d409e8.

Author: Enchufa2

ChangeLog

@@ -92,7 +92,7 @@
 2025-11-12 Iñaki Ucar <[email protected]>
 
 	* inst/include/Rcpp/macros/mask.h: Lay the ground for Rf_error masking
-	with a message at compilation time unless RCPP_NO_MASK_RF_ERROR is defined
+	with a warning at compilation time unless RCPP_NO_MASK_RF_ERROR is defined
 	* inst/include/RcppCommon.h: Include the previous file in the last place
 	* src/attributes.cpp: Use parentheses to protect call to Rf_error
 	* inst/tinytest/cpp/stack.cpp: Idem

inst/include/Rcpp/macros/mask.h

@@ -20,26 +20,14 @@
 #ifndef Rcpp_macros_mask_h
 #define Rcpp_macros_mask_h
 
-namespace Rcpp {
-namespace internal {
-
-template<typename... Args>
-void Rf_error(const char *format, Args... args) {
-    ::Rf_error(format, args...);
-#ifndef COMPILING_RCPP
-# pragma message "Use of Rf_error() instead of Rcpp::stop(). Calls \
+#ifndef RCPP_NO_MASK_RF_ERROR
+#define Rf_error(...) \
+    _Pragma("GCC warning \"Use of Rf_error() instead of Rcpp::stop(). Calls \
 to Rf_error() in C++ contexts are unsafe: consider using Rcpp::stop() instead, \
 or define RCPP_NO_MASK_RF_ERROR if this is a false positive. More info:\n\
  - https://github.com/RcppCore/Rcpp/issues/1247\n\
- - https://github.com/RcppCore/Rcpp/pull/1402"
-#endif
-}
-
-} // internal
-} // Rcpp
-
-#ifndef RCPP_NO_MASK_RF_ERROR
-#define Rf_error(...) Rcpp::internal::Rf_error(__VA_ARGS__)
+ - https://github.com/RcppCore/Rcpp/pull/1402\"") \
+    Rf_error(__VA_ARGS__)
 #endif
 
 #endif