Ticket #1117 (closed patch: fixed)

Opened 5 years ago

Last modified 5 years ago

PIR Book Typos

Reported by: rblasch Owned by: jkeenan
Priority: normal Milestone:
Component: docs Version: 1.6.0
Severity: low Keywords:
Cc: Language:
Patch status: new Platform:

Description (last modified by allison) (diff)

Here's a small patch with a few typos I've found in the PIR book. Not sure if I got everything right, though, please check.

The other fixes in this patch are good.

Attachments

pir_book.patch Download (2.8 KB) - added by rblasch 5 years ago.

Change History

Changed 5 years ago by rblasch

in reply to: ↑ description ; follow-up: ↓ 2   Changed 5 years ago by jkeenan

Replying to rblasch:

Here's a small patch with a few typos I've found in the PIR book. Not sure if I got everything right, though, please check.

Hmmm ... seems everybody's reading the PIR book this morning :-). See TT #116.

I'll manually correct the places where we have nothing but spelling errors and where there's no ambiguity. See r41928 and r41929. But in one location (docs/book/pir/ch04_variables.pod) Ron's suggested correction differs from mine, so I'll let wiser minds choose.

in reply to: ↑ 1 ; follow-up: ↓ 3   Changed 5 years ago by jkeenan

Replying to jkeenan:

Replying to rblasch:

Ron,

I guess my problem is that I don't understand what it means for something -- in this case, strings -- to be "support logical". If I did understand that, then your addition of a comma makes sense and I would apply that rather than deleting the two words as I suggested in TT #1116.

Thank you very much.

kid51

in reply to: ↑ 2   Changed 5 years ago by rblasch

Replying to jkeenan:

Replying to jkeenan:

Replying to rblasch:

Ron, I guess my problem is that I don't understand what it means for something -- in this case, strings -- to be "support logical". If I did understand that, then your addition of a comma makes sense and I would apply that rather than deleting the two words as I suggested in TT #1116. Thank you very much.

As I see it, if a number or string "supports logical," it can be used as term with the logical operators.

    $P0 = box 123
    $P1 = box "abc"
    $P2 = and $P0, $P1
    say $P2

Ron

follow-up: ↓ 5   Changed 5 years ago by allison

  • description modified (diff)

The phrase "support logical" should be deleted, it was a stray leftover from an edit to make the text clearer.

in reply to: ↑ 4   Changed 5 years ago by jkeenan

  • status changed from new to assigned
  • owner set to jkeenan

Replying to allison:

The phrase "support logical" should be deleted, it was a stray leftover from an edit to make the text clearer.

Handled in r41935; see TT #1116.

kid51

  Changed 5 years ago by jkeenan

  • status changed from assigned to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.