search

Daniel061 / Sophie6.1

AI from stumbling Baby to Sentient being.
GNU General Public License v3.0
1 stars 1 forks source link

Initialized the Sentence Token Position variable #17

Closed Ask149 closed 5 years ago

Ask149 commented 5 years ago

File Requested: Baby6.1/include/c_Sentence.h Purpose: Initializing the int_Sentence_Token_Position variable to '0', as it is referenced in the future once. Hi, I am understanding your project and I really wish to start contributing to your project. Could you please allow me to contribute to your project? Thanks!.

Ask149 commented 5 years ago

I'll surely try to maintain your code of conduct.

Daniel061 commented 5 years ago

Thank you for your interest in this project. I welcome your input. Remember, I would like to keep the code as readable as possible. Thank you again and Welcome to the project.

Ask149 commented 5 years ago

Sure, Thanks! I'll keep that in mind.

Daniel061 commented 5 years ago

Also, I read the purposed change. That's a good catch but the form of initialization is c++11. I have not yet enabled C++11 in my Code::Blocks. Do you see a benefit?

Ask149 commented 5 years ago

Also, I read the purposed change. That's a good catch but the form of initialization is c++11. I have not yet enabled C++11 in my Code::Blocks. Do you see a benefit?

I would suggest to upgrade to C++11 because later on you will probably need asynchronous proceesing and multithreading facilities which are available in C++11 onwards. Do you want me to scale down to C++ ?

Daniel061 commented 5 years ago

No, please do not scale down. I like the readability of the initialization method. I will enable C++11 on my Code::Blocks IDE. Thank You.