[Crate] Add better documentation for generic game implementation (#49)

src/games/mod.rs

@@ -116,11 +116,15 @@ use crate::protocols::{self, Protocol};
 use crate::GDResult;
 use std::net::{IpAddr, SocketAddr};
 
+/// Definition of a game
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq)]
 pub struct Game {
+    /// Full name of the game
     pub name: &'static str,
+    /// Default port used by game
     pub default_port: u16,
+    /// The protocol the game's query uses
     pub protocol: Protocol,
 }
 
@@ -130,6 +134,7 @@ mod definitions;
 #[cfg(feature = "game_defs")]
 pub use definitions::GAMES;
 
+/// Make a query given a game definition
 pub fn query(game: &Game, address: &IpAddr, port: Option<u16>) -> GDResult<protocols::GenericResponse> {
     let socket_addr = SocketAddr::new(*address, port.unwrap_or(game.default_port));
     Ok(match &game.protocol {

src/lib.rs

@@ -2,6 +2,7 @@
 //!
 //! # Usage example:
 //!
+//! ## For a specific game
 //! ```
 //! use gamedig::games::tf2;
 //!
@@ -12,12 +13,26 @@
 //! }
 //! ```
 //!
+//! ## Using a game definition
+//! ```
+//! use gamedig::games::{GAMES, query};
+//!
+//! let game = GAMES.get("tf2").unwrap(); // Get a game definition, the full list can be found in src/games/mod.rs
+//! let response = query(game, &"127.0.0.1".parse().unwrap(), None); // None will use the default port
+//! match response {
+//!     Err(error) => println!("Couldn't query, error: {}", error),
+//!     Ok(r) => println!("{:#?}", r),
+//! }
+//! ```
+//!
 //! # Crate features:
-//! Enabled by default: None
+//! Enabled by default: `game_defs`
 //!
 //! `serde` - enables json serialization/deserialization for all response types.
 //! <br> `no_games` - disables the included games support. <br>
-//! `no_services` - disables the included services support.
+//! `no_services` - disables the included services support. <br>
+//! `game_defs` - Include game definitions for programmatic access (enabled by
+//! default).
 
 pub mod errors;
 #[cfg(not(feature = "no_games"))]

src/protocols/gamespy/mod.rs

@@ -7,6 +7,7 @@ pub mod protocols;
 
 pub use protocols::*;
 
+/// Versions of the gamespy protocol
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq)]
 pub enum GameSpyVersion {
@@ -15,6 +16,7 @@ pub enum GameSpyVersion {
     Three,
 }
 
+/// Enum of versions and their ExtraResponse data
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq)]
 pub enum VersionedExtraResponse {

src/protocols/minecraft/types.rs

@@ -78,6 +78,7 @@ pub struct JavaResponse {
     pub server_type: Server,
 }
 
+/// Non-generic java response
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)]
 pub struct JavaExtraResponse {
@@ -146,6 +147,7 @@ pub struct BedrockResponse {
     pub server_type: Server,
 }
 
+/// Non-generic bedrock response
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)]
 pub struct BedrockExtraResponse {

src/protocols/quake/types.rs

@@ -24,6 +24,7 @@ pub struct Response<P> {
     pub unused_entries: HashMap<String, String>,
 }
 
+/// Non-generic quake response
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct ExtraResponse {

src/protocols/valve/types.rs

@@ -58,6 +58,7 @@ pub struct Response {
     pub rules: Option<HashMap<String, String>>,
 }
 
+/// Non-generic valve response
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Clone, PartialEq)]
 pub struct ExtraResponse {