hanshuebner / lmman

Lisp Machine Manual (XML Edition)
28 stars 5 forks source link

missing method names #3

Closed Devon7 closed 1 year ago

Devon7 commented 1 year ago

E.g., from the original text

Method: pathname :host
Method: pathname :device
Method: pathname :directory
Method: pathname :name
Method: pathname :type
Method: pathname :version

These return the components …

method names are missing in Mozilla Firefox 102.7.0esr

25.2 Pathname Components
⋮
Method  pathname
Method  pathname
Method  pathname
Method  pathname
Method  pathname
Method  pathname

These return the components …
hanshuebner commented 1 year ago

I think this is fixed, I'd appreciate your review and confirmation.

Devon7 commented 1 year ago

The original 1984 Sixth Edition Orangual page 550 text reads

24.2.2 Pathname Component Operations

:host Operation on pathname :device Operation on pathname :directory Operation on pathname :name Operation on pathname :type Operation on pathname :version Operation on pathname

with left justified :keyword method names and right justified "Operation on pathname" but Mozilla Firefox 102.7.0esr shows https://HansHuebner.github.io/lmman/pathnm.xml#directory%20(pathname) as

25.2 Pathname Components ⋮ pathname-host-method Method pathname-device-method Method pathname-directory-method Method pathname-name-method Method pathname-type-method Method pathname-version-method Method

with left justified XML names and right justified "Method" - not correct.

hanshuebner commented 1 year ago

I made another change that should more closely resemble what you've lined out. It would be nicer if the XSLT transformation would render these operations specially, but I'm not currently in the doing that. A pull request to fix it properly would be gladly accepted though.

Devon7 commented 1 year ago

The original reads :hostOperation on pathname

It would be OK to say :hostMethod on pathname

or a little confusing to say :hostMethod

or more confusing to say :host Operation on pathnameMethod

or most confusing to say :host Operation on pathnameMethod which is what it says now, but at least the method name is present, which is the essential.

♢ indicates maximum whitespace separating left-justified from right-justified text.

hanshuebner commented 1 year ago

Hi Devon,

I think it looks better now, after I have applied a more systematic fix to the XML files and corrected the rendering:

[image: image.png]

Meta-methods are still not displayed correctly, though. Can you have a look at the streams chapter (which defines a lot of metamethods) and tell me how those should look like? I suspect it would be similar to methods, but does it say "meta-operation"?

Here's the current, wrong rendition:

[image: image.png]

Thank you, Hans

Devon7 commented 1 year ago

SIGH Taking the plunge deeper into yak-shaving until the next paying customer comes along! Orangual page 461 reads 22.3.2 Standard Input Stream Operations

:tyi &optional eofOperation on streams The stream inputs one character ☆☆☆ the :tyi operation does not echo

Firefox sees stream &optional eofMeta-Method The stream inputs one character ☆☆☆ the :tyi operation does not echo

omitting the section header and operation/"method" names therein.

☆ indicates horizontal whitespace separating left-justified from right-justified text. ☆☆☆ indicates elided text in the same paragraph.

Devon7 commented 1 year ago

This issue is incorrectly marked closed - method names are still missing, as shown above.