Fork me on GitHub
  May 25, 2018, 08:48:09 am
Welcome, Guest. Please login or register.
Did you miss your activation email?

Login with username, password and session length
   Home   Help Search Login Register  
Pages: [1]
  Print  
Author Topic: Basic questions about the development process  (Read 1542 times)
louisdx
Newbie
*
Posts: 12


View Profile
« on: April 01, 2011, 12:47:41 am »

Hi - I just had a quick browse through your source, and I was wondering how one could contribute. Do you accept patch submissions, or would it be at all useful if I post small comments?

A quick question to begin: How complete is your knowledge of the network protocol, and where is it documented?

OK. Just looking at src/mineserver.h/cpp I have some very minor suggestions:

  • mineserver.h doesn't seem to use the <set> and <iostream> headers; those can go.
  • mineserver.cpp: in main(), you could move the declaration of "arg" inside the loop: "std::string arg(argv)". On the other hand, using getopt.h would probably be an even better approach (one could include a portable version for non-POSIX targets).
  • mineserver.cpp: With std::tr1 you have access to a few very good standardized random number generators, might be worth checking out.
  • Across other header files, there's a mix between deprecated and modern C headers, <string.h> vs. <cstring> etc. Might be nice to make that uniform.

It'd be great to see this project succeed!
Logged
Ligustah
Newbie
*
Posts: 42


View Profile
« Reply #1 on: April 01, 2011, 01:19:47 pm »

I think most (all?) contributions are done via pull requests on Github.
Logged
louisdx
Newbie
*
Posts: 12


View Profile
« Reply #2 on: April 01, 2011, 02:45:58 pm »

I think most (all?) contributions are done via pull requests on Github.

Thank you! I've never used git before (svn user myself), but now I have a reason to learn it :-) I just set up an account, made a custom key pair and forked off.
Logged
Pages: [1]
  Print  
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1.21 | SMF © 2015, Simple Machines Valid XHTML 1.0! Valid CSS!