From a2ecd53f5822ae07fabfc7d2fac7fa929eb7f795 Mon Sep 17 00:00:00 2001 From: sterni Date: Fri, 4 Feb 2022 12:15:57 +0100 Subject: [PATCH] chore: clean up some obsolete TODOs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * //3p/nix: probably not worth investing time into this anymore * //users/sterni/emacs: The emoji problem disappeared by itself with a newer emacs version, however a different one remains… * //web/panettone: If we ever want to change the behavior, we should just decide the behavior statically instead of using conditions and restarts, as we only call it in one place, so making different decisions depending on call sites is not really a use case we have. Change-Id: Iff9d439ce356db41ce34d690fb7b6a01822022fa Reviewed-on: https://cl.tvl.fyi/c/depot/+/5223 Tested-by: BuildkiteCI Reviewed-by: tazjin Autosubmit: sterni --- third_party/nix/src/libexpr/primops.cc | 1 - users/sterni/emacs/init.el | 2 +- web/panettone/src/inline-markdown.lisp | 3 --- 3 files changed, 1 insertion(+), 5 deletions(-) diff --git a/third_party/nix/src/libexpr/primops.cc b/third_party/nix/src/libexpr/primops.cc index 710d214ce..f196c5ed7 100644 --- a/third_party/nix/src/libexpr/primops.cc +++ b/third_party/nix/src/libexpr/primops.cc @@ -1370,7 +1370,6 @@ static void prim_functionArgs(EvalState& state, const Pos& pos, Value** args, Value& v) { state.forceValue(*args[0]); if (args[0]->type == tPrimOpApp || args[0]->type == tPrimOp) { - // TODO(sterni): return set of formal arguments for fetch* primops state.mkAttrs(v, 0); return; } diff --git a/users/sterni/emacs/init.el b/users/sterni/emacs/init.el index 1cd9e9c36..c75050b2c 100644 --- a/users/sterni/emacs/init.el +++ b/users/sterni/emacs/init.el @@ -1,5 +1,5 @@ ;; Set default font and fallback font via set-fontset-font -;; TODO(sterni): Investigate non-emoji representation of some glyphs +;; TODO(sterni): Investigate why ZWJ sequences aren't shaped properly (let ((mono-font "Bitstream Vera Sans Mono-12") (emoji-font "Noto Color Emoji-12")) (setq default-frame-alist `((font . ,mono-font))) diff --git a/web/panettone/src/inline-markdown.lisp b/web/panettone/src/inline-markdown.lisp index fc5f77584..e49293519 100644 --- a/web/panettone/src/inline-markdown.lisp +++ b/web/panettone/src/inline-markdown.lisp @@ -124,7 +124,4 @@ ; only eliminates the slight use case for nesting :em inside :del, but ; shouldn't be too bad. As a side effect this is the precise behavior ; we want for :code. - ; - ; TODO(sterni): maybe bring back the restart-based system which allowed - ; to skip nested tokens if desired. (t (write-string (who:escape-string tok-str) target)))))