-
-
Notifications
You must be signed in to change notification settings - Fork 343
Anatomy of an Array/Memory Layout <-- maybe make it clearer? #96
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
Comments
Yes, this is the perfect place to report the issue. Thanks a lot and I think you're right that there's a problem. Could you make a PR to fix it? |
I will... As soon as I learn how to do it properly :) |
No problem. Just ask me if you need some help (if this is your first pull request, it might appear a bit complex) |
The easiest way, when the change is small is to browse the file on my repository and to "edit" it (the pen button on upper right). Since you do not have rights to modify the file, GitHub will propose you automatically to make a PR for proposing your change. The other way is more complex but not that much:
And you're mostly done. If you go on GitHub gain, you'll see your last commit and GitHub will propose you to make a PR. If it's not clear, just tell me. I remember that I totally messed up my first PR a few years ago and someone helped me a lot) |
Hi,
sorry I'm not 100% sure this is the right place to suggest that, but the following code can result a little unclear because you reference a variable
index
which is not previously defined:Then, when double-checking whether the idea works, I think you mix up variable names a little:
Not a big deal but I think a 'fusion' of the two snippets into one would be clearer, as direct implementation of what you propose does not work.
Best,
M
The text was updated successfully, but these errors were encountered: