From: Kevin Carhart <kevin@carhart.net>
To: Edbrowse-dev@lists.the-brannons.com
Subject: [Edbrowse-dev] (something other than) whitespace nodes
Date: Fri, 25 Aug 2017 21:25:42 -0700 (PDT) [thread overview]
Message-ID: <alpine.LRH.2.03.1708252108110.27415@carhart.net> (raw)
In-Reply-To: <20170725185648.eklhad@comcast.net>
Thanks for pointing this out. I guess I took something overly literal
that is not a part of the generic principle they're getting at in the
test. Clearly node-ifying every '\n' in every web page isn't common or
important or we would have hit it previously.. I could have keyed in to
this fact sooner. Oh well. I was only in tidy for a short time, and the
exploration seems useful anyhow.
On Fri, 25 Aug 2017, Karl Dahlke wrote:
>> I haven't been able to get additional nodes created out of newlines
>
> Not sure how hard we should work on this, or even if we want it, just to pass an acid test.
> It probably has no bearing in the real world, and who wants all those empty nodes cluttering up the tree?
> For now I think we should just delete or comment out line 227 in the acid test file,
> it's just understood that this line is nulled out, then test 0 should pass and we move on.
> Let's get the value out of the acid tests without becoming obsessed over them.
> That's my gut feeling right now.
>
> Karl Dahlke
> _______________________________________________
> Edbrowse-dev mailing list
> Edbrowse-dev@lists.the-brannons.com
> http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev
>
--------
Kevin Carhart * 415 225 5306 * The Ten Ninety Nihilists
next prev parent reply other threads:[~2017-08-26 4:25 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-19 15:38 [Edbrowse-dev] acid[0] Karl Dahlke
2017-08-19 22:53 ` Kevin Carhart
2017-08-19 23:08 ` Karl Dahlke
2017-08-19 23:33 ` Kevin Carhart
2017-08-20 0:00 ` Karl Dahlke
2017-08-20 0:37 ` Kevin Carhart
2017-08-20 14:33 ` Karl Dahlke
2017-08-20 20:00 ` Kevin Carhart
2017-08-20 20:08 ` [Edbrowse-dev] getAttributeNode / setAttributeNode Kevin Carhart
2017-08-20 20:24 ` Karl Dahlke
2017-08-20 20:56 ` Kevin Carhart
2017-08-20 21:59 ` Kevin Carhart
[not found] ` <20170721105041.eklhad@comcast.net>
2017-08-21 19:11 ` Kevin Carhart
2017-08-21 20:01 ` Karl Dahlke
2017-08-24 9:54 ` Kevin Carhart
2017-08-24 9:57 ` Kevin Carhart
2017-08-25 8:19 ` Kevin Carhart
2017-08-25 22:09 ` [Edbrowse-dev] whitespace nodes Kevin Carhart
2017-08-25 22:56 ` Karl Dahlke
2017-08-26 4:25 ` Kevin Carhart [this message]
2017-09-02 9:03 ` [Edbrowse-dev] (something other than) " Adam Thompson
2017-09-02 15:42 ` Karl Dahlke
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=alpine.LRH.2.03.1708252108110.27415@carhart.net \
--to=kevin@carhart.net \
--cc=Edbrowse-dev@lists.the-brannons.com \
/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 NNTP newsgroup(s).