Skip to content

strings#5

Merged
Book-reader merged 6 commits intoc3lang:mainfrom
neokeld:strings-doc
Feb 23, 2026
Merged

strings#5
Book-reader merged 6 commits intoc3lang:mainfrom
neokeld:strings-doc

Conversation

@neokeld
Copy link
Copy Markdown
Contributor

@neokeld neokeld commented Feb 2, 2026

ZString, String, DString and an explanation about char** because I had a problem with it.

Copy link
Copy Markdown
Member

@Book-reader Book-reader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this is good. I'm not sure about the char** part as it currently is though, what was your reason for adding it? It seems like it's more about working with arrays than strings (and a minor nitpick is that usually a pointer to an array of C strings would be represented as ZString* instead of char**, but that's easy to change).

@lerno
Copy link
Copy Markdown
Contributor

lerno commented Feb 7, 2026

Perhaps make code formatted like the C3 stdlib.

@Book-reader
Copy link
Copy Markdown
Member

I've removed the char** example since there is already an example of converting an array to a pointer in the arrays page, if the example there is not enough I can improve it (the arrays page could use some improvement anyway).

@Book-reader Book-reader merged commit b583691 into c3lang:main Feb 23, 2026
@Book-reader
Copy link
Copy Markdown
Member

Thank you!

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

Successfully merging this pull request may close these issues.

3 participants