emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[elpa] externals/csharp-mode 803a362 203/459: Move log functions back do


From: ELPA Syncer
Subject: [elpa] externals/csharp-mode 803a362 203/459: Move log functions back down where they were.
Date: Sun, 22 Aug 2021 13:59:27 -0400 (EDT)

branch: externals/csharp-mode
commit 803a36259c2e849bbb080d8f6635a81a30a5ee49
Author: Jostein Kjønigsen <jostein@kjonigsen.net>
Commit: Jostein Kjønigsen <jostein@kjonigsen.net>

    Move log functions back down where they were.
    
    For whatever reason, moving them up caused the following issue:
    https://github.com/josteink/csharp-mode/issues/79.
    
    Moving them back down seems to fix it.
---
 csharp-mode.el | 46 +++++++++++++++-------------------------------
 1 file changed, 15 insertions(+), 31 deletions(-)

diff --git a/csharp-mode.el b/csharp-mode.el
index b1a36b4..f73d182 100644
--- a/csharp-mode.el
+++ b/csharp-mode.el
@@ -446,37 +446,6 @@ Most other csharp functions are not instrumented.
 ;; csharp-mode utility and feature defuns
 ;; ==================================================================
 
-(defun csharp-time ()
-  "returns the time of day as a string.  Used in the `csharp-log' function."
-  (substring (current-time-string) 11 19)) ;24-hr time
-
-
-;; essentially the same as (progn), but this this is required to avoid
-;; byte-compilation warnings due to some forms referencing this
-;; getting expanded during compilation.
-(eval-when-compile
-  (defun csharp-log (level text &rest args)
-    "Log a message at level LEVEL.
-If LEVEL is higher than `csharp-log-level', the message is
-ignored.  Otherwise, it is printed using `message'.
-TEXT is a format control string, and the remaining arguments ARGS
-are the string substitutions (see `format')."
-    (if (<= level csharp-log-level)
-        (let* ((msg (apply 'format text args)))
-          (message "C# %s %s" (csharp-time) msg)))))
-
-;; nasty hack to silence compile-time warnings and runtime-warnings.
-;; exact copy of defun above.
-(defun csharp-log (level text &rest args)
-  "Log a message at level LEVEL.
-If LEVEL is higher than `csharp-log-level', the message is
-ignored.  Otherwise, it is printed using `message'.
-TEXT is a format control string, and the remaining arguments ARGS
-are the string substitutions (see `format')."
-  (if (<= level csharp-log-level)
-      (let* ((msg (apply 'format text args)))
-        (message "C# %s %s" (csharp-time) msg))))
-
 (defun csharp--at-vsemi-p (&optional pos)
   "Determines if there is a virtual semicolon at POS or point.
 It returns t if at a position where a virtual-semicolon is.
@@ -2408,6 +2377,21 @@ your `csharp-mode-hook' function:
 ;; end of c# code-doc insertion magic
 ;; ==================================================================
 
+(defun csharp-time ()
+  "returns the time of day as a string.  Used in the `csharp-log' function."
+  (substring (current-time-string) 11 19)) ;24-hr time
+
+
+(defun csharp-log (level text &rest args)
+  "Log a message at level LEVEL.
+If LEVEL is higher than `csharp-log-level', the message is
+ignored.  Otherwise, it is printed using `message'.
+TEXT is a format control string, and the remaining arguments ARGS
+are the string substitutions (see `format')."
+  (if (<= level csharp-log-level)
+      (let* ((msg (apply 'format text args)))
+        (message "C# %s %s" (csharp-time) msg))))
+
 ;; ==================================================================
 ;; C#-specific optimizations of cc-mode funcs
 ;; ==================================================================



reply via email to

[Prev in Thread] Current Thread [Next in Thread]