docsrc patch for pages/testing.in
(1) By brickviking on 2023-04-19 01:04:58 [source]
I noticed this possible omission in docsrc revision c69640a74c with pages/testing.in:
Index: pages/testing.in
==================================================================
--- pages/testing.in
+++ pages/testing.in
@@ -859,7 +859,7 @@
not only that every branch instruction both jumps and falls through but also
that every branch is doing useful work and that the test suite is able
to detect and verify that work. When a branch is found that does not
-make a difference in the output, that suggests that the code associated
+make a difference in the output, that suggests that the code associated with
the branch can be removed (reducing the size of the library and perhaps
making it run faster) or that the test suite is inadequately testing the
feature that the branch implements.
I hope there's no problem with the patch posted this way. As a side note: Man, I like the preview feature, it's caught about three of my mistakes.
Cheers, brickviking
(Post 15)
(2) By Stephan Beal (stephan) on 2023-04-19 01:16:23 in reply to 1 [link] [source]
I hope there's no problem with the patch posted this way.
Absolutely not. That will be fixed momentarily.
As a side note: Man, I like the preview feature, it's caught about three of my mistakes.
Yeah, now if we could only get more people to take advantage of it ;).
(3) By Larry Brasfield (larrybr) on 2023-04-19 01:17:54 in reply to 1 [link] [source]
Thanks for the pick-up. This grammo has been cured.
I hope there's no problem with the patch posted this way.
No problem, but just the affected line, before and after, is just as adequate.
(4) By brickviking on 2023-04-19 01:39:19 in reply to 3 [link] [source]
... but just the affected line, before and after, ...
Fossil diff gave me 11 lines by default, though if I dipped into the options I have no doubt I'd find some way of reducing that to a more reasonable subset like I did.
As a wee pet project I rendered these pages then saved them to a new fossil, and tweaked the default_csp to allow the inline scripts to run. It's nice to be able to access the documents from inside fossil's own interface and just have everything Workâ˘. The process was relatively simple once I did the relevant reading.
The grammo hase been cured.
You might see a couple more of these "grammos" with patches in the near future as I spot them.
Cheers again, brickviking
(Post 16)
(5) By Larry Brasfield (larrybr) on 2023-04-19 01:41:36 in reply to 4 [link] [source]
You might see a couple more of these "grammos" with patches in the near future as I spot them.
They are always welcome. However, unless you are very ill or prone to walking in front of moving busses, please feel welcome to batch such changes.
(6) By Stephan Beal (stephan) on 2023-04-19 01:45:08 in reply to 4 [link] [source]
Fossil diff gave me 11 lines by default, though if I dipped into the options I have no doubt I'd find some way of reducing that to a more reasonable subset like I did.
The -c flag:
$ fossil diff -c 2
where (for example) 2 is whatever number of lines of (-c)ontext you want.