unofficial mirror of help-gnu-emacs@gnu.org
 help / color / mirror / Atom feed
From: Jean Louis <bugs@gnu.support>
To: Michael Heerdegen <michael_heerdegen@web.de>
Cc: help-gnu-emacs@gnu.org
Subject: Re: Adding String to Beginning of Lines
Date: Tue, 10 Nov 2020 22:12:33 +0300	[thread overview]
Message-ID: <X6rmIUwQoEQN2ed8@protected.rcdrun.com> (raw)
In-Reply-To: <87tutxb7yz.fsf@web.de>

* Michael Heerdegen <michael_heerdegen@web.de> [2020-11-10 17:30]:
> Jean Louis <bugs@gnu.support> writes:
> 
> > (let ((regex (or regex default))))
>                    ^^^^^
> 
> That might possible need to be
> 
>   (string-empty-p regex)
> 
> depending on how the contents of the variable had been read.

Thank you.

User can choose to have "" regex as also plain function allows is.

(string-match "" "something")

If it is "" it is not nil, it would be chosen.

If it is nil it would not be chosen.

Is that right? Do you think it is necessary more than just `or'?

Afterthoughts:

I do not link that string-empty-p function

(string-empty-p nil) => nil

It is not logical to me. I was even surprised to find it just
yesterday because I had my own function.

Of course that NIL is not a string, but it is also not a string to
give NIL if it is empty (I know this sentence may be confusing) as
that means that NIL is not empty string.

If string is NIL I expect that string-empty-p on that string tells me
TRUE.

(defun rcd-string-empty-p (s)
  "Returns T if string is empty or NIL"
  (let ((s (if (null s) "" s)))
    (if (stringp s)
	(if (zerop (length s)) t nil))))

(rcd-string-empty-p nil) => T

That is my expectation in many programs where I am handling the string
which can be also NIL.

I was always loading files and then first time I installed it as
package that uses subr-x (I think function comes from there) so my
personal function string-empty-p collided with subr-x function and
suddenly my program did not behave well. Yesterday I have renamed my
function.



  reply	other threads:[~2020-11-10 19:12 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-10 10:43 Adding String to Beginning of Lines Christopher Dimech
2020-11-10 11:10 ` tomas
2020-11-10 11:19   ` Christopher Dimech
2020-11-10 11:29   ` Christopher Dimech
2020-11-10 11:54     ` Michael Heerdegen
2020-11-10 11:58       ` Christopher Dimech
2020-11-10 12:07         ` Michael Heerdegen
2020-11-10 12:03       ` Christopher Dimech
2020-11-10 12:16         ` Michael Heerdegen
2020-11-10 13:40           ` Christopher Dimech
2020-11-10 14:06             ` Jean Louis
2020-11-10 14:29               ` Michael Heerdegen
2020-11-10 19:12                 ` Jean Louis [this message]
2020-11-10 19:42                   ` Michael Heerdegen
2020-11-10 21:22                     ` Jean Louis
2020-11-11 10:59                       ` Michael Heerdegen
2020-11-17 16:06                         ` Jean Louis
2020-11-17 16:28                           ` Michael Heerdegen
2020-11-10 22:31                     ` Stefan Monnier
2020-11-10 23:00                       ` Drew Adams
2020-11-11 21:37                         ` Michael Heerdegen
2020-11-10 14:16             ` Michael Heerdegen
2020-11-10 14:38               ` Christopher Dimech
2020-11-10 14:49                 ` Michael Heerdegen
2020-11-10 17:52                   ` Christopher Dimech
2020-11-10 18:03                     ` Michael Heerdegen
2020-11-10 18:25                       ` Christopher Dimech
2020-11-10 18:49                         ` Michael Heerdegen
2020-11-10 18:56                           ` Christopher Dimech
2020-11-10 19:02                             ` Michael Heerdegen
2020-11-10 19:29                               ` Christopher Dimech
2020-11-10 20:07                                 ` Michael Heerdegen
2020-11-10 22:45             ` tomas
2020-11-11 11:16               ` Michael Heerdegen
2020-11-11 13:22                 ` Christopher Dimech
2020-11-11 13:56                   ` Michael Heerdegen
2020-11-11 14:07                   ` tomas
2020-11-11 14:49                     ` Christopher Dimech
2020-11-11 17:20                 ` Christopher Dimech
2020-11-11 18:12                   ` Michael Heerdegen
2020-11-11 18:25                     ` Stefan Monnier
2020-11-11 18:35                     ` Christopher Dimech
2020-11-10 14:03         ` Jean Louis
2020-11-10 11:42 ` Michael Heerdegen
2020-11-10 18:47   ` H. Dieter Wilhelm
2020-11-12 19:01 ` Dante Catalfamo
2020-11-14 22:09   ` Jamie Beardslee
2020-11-14 23:17     ` Stefan Monnier
2020-11-14 23:39       ` Jamie Beardslee
2020-11-15  9:12       ` tomas

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=X6rmIUwQoEQN2ed8@protected.rcdrun.com \
    --to=bugs@gnu.support \
    --cc=help-gnu-emacs@gnu.org \
    --cc=michael_heerdegen@web.de \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).