Skip to content

Conversation

@hobovsky
Copy link
Contributor

@hobovsky hobovsky commented Jan 10, 2021

Dedicated issue: #230
Preview: here

As I explained to Unnamed below:

[...] my intention [behind this article] is not to create C programmers guide, I am just looking for a way to transfer some ideas to C authors and translators, to get rid of ideas like requirement of mallocing a 3-element array to return hours, minutes, and seconds. Or returning Position**, where each entry is... a pointer to an individually malloced, single Position :( And there are even worse. I just thought about collecting some ideas and throw them at the authors, so they could pick something suitable for their kata. It's meant to be CW- and kata-oriented, and not an industry grade HOWTO ;) That's why I do not want to make things too complex and too detailed. Maybe I already have, maybe this paragraph, or this whole article, is simply not necessary?

Let me know what you think about this article, and even if you think it's not necessary or too detailed, it was a real fun writing it ;)

@hobovsky hobovsky requested a review from ggorlen January 14, 2021 12:40
@Steffan153
Copy link
Contributor

Steffan153 commented Jan 14, 2021

🙄 2 new commits while I'm reviewing...
Edit: Had to delete several outdated suggestions, will redo them when you finish these.

hobovsky and others added 2 commits January 14, 2021 15:59
Co-authored-by: Steffan <40404519+Steffan153@users.noreply.github.com>
@hobovsky hobovsky requested a review from Steffan153 January 14, 2021 15:09
hobovsky and others added 2 commits January 14, 2021 16:21
Co-authored-by: Steffan <40404519+Steffan153@users.noreply.github.com>
@hobovsky hobovsky requested a review from Steffan153 January 15, 2021 00:04
Co-authored-by: Donald Sebastian Leung <donaldsebleung@gmail.com>
Co-authored-by: Steffan <40404519+Steffan153@users.noreply.github.com>
@hobovsky hobovsky requested a review from kazk January 15, 2021 17:06
Co-authored-by: Donald Sebastian Leung <donaldsebleung@gmail.com>
@hobovsky hobovsky merged commit 0a0a83e into master Jan 16, 2021
@hobovsky hobovsky deleted the c-memory-management branch January 16, 2021 11:27
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.

Memory management patterns for C kata

7 participants