Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

documentation #42

Closed
zenon opened this issue Jan 9, 2020 · 3 comments
Closed

documentation #42

zenon opened this issue Jan 9, 2020 · 3 comments

Comments

@zenon
Copy link

zenon commented Jan 9, 2020

Hello,

today I took the pleasure to play with min. Thank you for this, it's cool!

As that's the only moment to really read documentation, I did note about spelling issues, strangenesses and whatever. I hope this will help.

Kind regards, and thank you!

zenon


Notes about https://h3rald.com/min/Min_DeveloperGuide.htm
Quotes in bold.

2.2
To exit min shell, press CTRL+C or type exit and press ENTER.

z: For exit to work, there needs to be an int on the stack. Suggested formulations:

  • .. or type 0 exit and press ENTER.
  • .. or use exit. [with link to the function]

3.
..
Then, another list containing two symbols (dup and ) is pushed on the stack. This constitutes a quoted program which, when executed duplicates (dup) the first element on the stack and then multiplies () the two elements together.

z: It can be confusing whether the parens are mentioned or used. I mean, you talk about (dup *) [here i mention the parentheses :-)], and then you use several parentheses. Suggestion:

Then, another list containing two symbols, i.e. dup and *, is pushed on the stack. This constitutes a quoted program which, when executed duplicates the first element on the stack -this is done by dup- and then multiplies -with *- the two elements together.


Parentheses are grouped together one or more elements, so that they are treated as a single element and they are not evaluated immediately.

z: are grouped together -> are used to group together


Symbols can be used to perform operations on the whole stack.

z: Not clear. Does this introduce the concept of symbol?
Do you mean: Symbols, also called words, refer to functions that operate on ..
?

3.1 Data Types

z: the type boolean seems to exist, but is missing.

z: The link to the dict module is broken


3.2.1
..
The synbol**
z: (spelling) synbol -> symbol


.. while the symbol apply can be used to dequote a quotation by pushing its elements on a separate stack.

z: this needs some explanation. until now we only heard of one stack.


3.3 Operators
..
Besides symbols, min provides a set of predefined sigils for commonly-used symbols.

z: I suggest an addition, like:
Besides symbols, min provides a set of predefined sigils as abbreviation for commonly-used symbols.

z: In the sigil list, the links to dget, dset, dhas? are broken


*3.4 Definitions
Being a concatenative language, min does not really need named parameters or variables: simbols just pop..

z: (spell) simbols -> symbols


3.4.1 Lexical scoping and binding

z: broken link to wikipedia


4.2
z: my windows computer doesn't have a HOMEPROFILE environment variable. It has USERPROFILE instead.

I didn'tread the reference yet. Except http; it'll get an extra issue.

@h3rald
Copy link
Owner

h3rald commented Jan 9, 2020

Excellent!!!! I have a terrible habit not to re-read what I write, and even when I do I just don't spot things sometimes.

I'll go through all this and improve the docs. Thanks again!

Out of curiosity, are you using the pre-built libraries or are you compiling min from source?

@zenon
Copy link
Author

zenon commented Jan 10, 2020

Good Morning!

I feel the need to add that nim is one of the rather well documented projects :-)

Prebuild unter windows.

Under Linux (more specific Debian Buster in its Windows Subssystem incarnation, don't ask :-) ) the prebuild did not work; some issues with the version of libc.so, it seems. So I built it myself, and that worked smoothly.

Kind Greetings!, z.

h3rald added a commit that referenced this issue Jan 10, 2020
@h3rald
Copy link
Owner

h3rald commented Jan 10, 2020

Everything should be fixed now, thanks again!

@h3rald h3rald closed this as completed Jan 10, 2020
h3rald added a commit that referenced this issue Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants