Skip to content
This repository has been archived by the owner on Nov 27, 2024. It is now read-only.

Changed README and LICENSE year #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Changed README and LICENSE year #26

wants to merge 3 commits into from

Conversation

bugravirgomercury
Copy link

Tried my best to contribute to this project. I have a couple of tips for you.

  • Consider using CMake to configure and build your project, which is more beginner friendly.
  • Be careful of which system headers you're using, as not all functions defined by them can be run in kernel space. You may need to reimplement them.
  • Try to move definitions into header files (.h) then include them in source files (.c) instead.
  • You'll need to configure include paths accordingly.
  • Check for syntax errors much more often, or use an integrated development environment to do it for you. The latter will give suggestions whilst you're writing code, increasing efficiency.

And lastly,

- #define <typename> <type>
+ typedef <typename> <type>;

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant