all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Matt <matt@excalamus.com>
To: "Ihor Radchenko" <yantar92@posteo.net>
Cc: "emacs-orgmode" <emacs-orgmode@gnu.org>
Subject: Re: [PATCH] ob-shell: consistent prefix
Date: Sat, 20 Jan 2024 17:03:00 +0100	[thread overview]
Message-ID: <18d279c718b.d21f7c10365942.3187894427394455251@excalamus.com> (raw)
In-Reply-To: <87ttndl8wk.fsf@localhost>

I'm going to defer these changes for now.  It seems like it would be easier to handle renaming one function/variable at a time.  Renaming requires creating an alias mapping between the old name and the new one.  Any subsequent refactoring would require updating the mapping.  It seems better to do any non-naming refactoring and then handle aliasing and obsoleting in order to minimize the number of changes that need to happen.  Also, renaming now invalidates any references, like in my notes or documentation.

I have questions regarding development that I'll ask in a different subject thread.

--
Matt Trzcinski
Emacs Org contributor (ob-shell)
Learn more about Org mode at https://orgmode.org
Support Org development at https://liberapay.com/org-mode



      reply	other threads:[~2024-01-20 16:03 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-15 21:12 [PATCH] ob-shell: consistent prefix Matt
2024-01-16 13:12 ` Ihor Radchenko
2024-01-20 16:03   ` Matt [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18d279c718b.d21f7c10365942.3187894427394455251@excalamus.com \
    --to=matt@excalamus.com \
    --cc=emacs-orgmode@gnu.org \
    --cc=yantar92@posteo.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.