user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
From: Eric Wong <e@yhbt.net>
To: Kyle Meyer <kyle@kyleam.com>
Cc: meta@public-inbox.org
Subject: Re: [PATCH] doc: add technical/whyperl
Date: Wed, 8 Apr 2020 22:26:07 +0000	[thread overview]
Message-ID: <20200408222607.GA4741@dcvr> (raw)
In-Reply-To: <871roykdv5.fsf@kyleam.com>

Kyle Meyer <kyle@kyleam.com> wrote:
> No substantial comments, just some typos spotted while reading through.

Thanks :>

> Eric Wong <e@yhbt.net> writes:
> 
> > +As always, comments and corrections and additions welcome at
> 
> s/welcome/are welcome/ ?

Umm... I guess?  Would omitting "are" would only be valid
if there were a single item?

	As always, $FOO welcome at

Upon a second read of the original, having "and" twice doesn't
read well to me.  So final form should probably be:

	As always, comments, corrections and additions are welcome at

> > +* Predictable performance
> > +
> > +  While Perl is neither fast or memory-efficient, its
> 
> s/or/nor/

I had to look that up, but yes, "nor" belongs there.

> > +  performance and memory use are predictable and does not
> 
> s/does/do/
> 
> > +  require GC tuning by the user.

I think "does" reads better, there, but the sentence runs on
for too long.  I don't think the GC part needs to be there(*)
Perhaps just:

	While Perl is neither fast nor memory-efficient, its
	performance and memory use are predictable.

(*) I think memory management requires a standalone document.

> > +  Low hardware requirements increases the reach of our software
> 
> s/increases/increase/

Yes.  I think my original wording was more about a continual
process (which it is):

	Lowering hardware requirements increases

But I edited it since requirements have always been fairly low.

> > +  Perl modules and bindings to common libraries such as
> > +  SQLite and Xapian are already distributed by many
> > +  GNU/Linux distros and BSD ports.
> > +
> > +  There should be no need to rely on language-specific
> > +  package managers such as cpan(1), those systems increase
> 
> s/, those/.  Those/

I was trying to decide between them.  I thought a separate
sentence would make it less obvious "those" was referring to
package managers.  In retrospect there's no ambiguity since
the entire paragraph is about language-specific package
managers.

> > +* Compactness and terseness
> > +
> > +  Less code generally means less bugs.  We try to avoid the
> 
> s/less bugs/fewer bugs/

I was thinking repeating "less" might be more memorable.
As in: "mo money, mo problems"; "less code, less bugs" :>
But yeah, "fewer" is probably better in a full sentence.

> > +* Lightweight threading
> > +
> > +  While lightweight threading implementations are
> > +  convenient, they tend to be significantly heavier than a
> > +  pure event-loop systems (or multi-threaded event-loop
> 
> s/a pure/pure/

Oops, yes.

  reply	other threads:[~2020-04-08 22:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-07  9:49 [PATCH] doc: add technical/whyperl Eric Wong
2020-04-08  0:41 ` Kyle Meyer
2020-04-08 22:26   ` Eric Wong [this message]
2020-04-09  1:14     ` Kyle Meyer
     [not found] ` <87r1wyjlop.fsf@vuxu.org>
2020-04-08 22:17   ` Eric Wong
2020-04-08 22:37     ` Leah Neukirchen

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: http://public-inbox.org/README

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

  git send-email \
    --in-reply-to=20200408222607.GA4741@dcvr \
    --to=e@yhbt.net \
    --cc=kyle@kyleam.com \
    --cc=meta@public-inbox.org \
    /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 public inbox

	https://80x24.org/public-inbox.git

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).