fix build

gears/src/baseapp/mod.rs

@@ -110,16 +110,18 @@ impl<DB: Database, PSK: ParamsSubspaceKey, H: ABCIHandler, AI: ApplicationInfo>
 
     fn run_query(&self, request: &RequestQuery) -> Result<Bytes, QueryError> {
         //TODO: request height u32
-        let version: NonZero<u32> = NonZero::new(
+        let version = NonZero::new(
             request
                 .height
                 .try_into()
                 .map_err(|_| QueryError::InvalidHeight)?,
-        )
-        .ok_or(QueryError::InvalidHeight)?;
+        );
 
         let store = self.multi_store.read().expect(POISONED_LOCK);
-        let ctx = QueryContext::new(QueryMultiStore::new(&*store, Some(version))?, version.get())?;
+        let ctx = QueryContext::new(
+            QueryMultiStore::new(&*store, version)?,
+            version.map(|this| this.get()).unwrap_or_default(),
+        )?;
 
         self.abci_handler
             .query(&ctx, request.clone())

gears/src/baseapp/mode/check.rs

@@ -16,6 +16,8 @@ use crate::{
     },
 };
 
+/// Specific to `check_tx` ABCI method.
+///
 /// Mode to validate a transaction before letting
 /// them into local mempool. This mode doesn't execute a
 /// transaction only runs ante checks on them.

gears/src/baseapp/mode/deliver.rs

@@ -14,6 +14,8 @@ use crate::{
     types::tx::raw::TxWithRaw,
 };
 
+/// Specific to `deliver_tx` ABCI method.
+///
 /// Mode to execute transactions. This mode still run
 /// ante checks, but you may skip execution by checking
 /// `bool` flag in ante method. Futhermore this module

gears/src/baseapp/state.rs

@@ -8,6 +8,8 @@ use crate::{
 
 use super::mode::{check::CheckTxMode, deliver::DeliverTxMode};
 
+/// Structure to hide state logic from application
+/// and sync state between different modes during blocks
 #[derive(Debug)]
 pub struct ApplicationState<DB, AH: ABCIHandler> {
     pub(super) check_mode: CheckTxMode<DB, AH>,
@@ -26,6 +28,8 @@ impl<DB: Database, AH: ABCIHandler> ApplicationState<DB, AH> {
         }
     }
 
+    /// Update gas meter for block metering. This method should be called in each `begin_block`
+    /// to update meter according to [crate::baseapp::params::BlockParams]
     pub fn replace_meter(&mut self, max_gas: Gas) {
         match max_gas {
             Gas::Infinite => {
@@ -64,6 +68,10 @@ impl<DB: Database, AH: ABCIHandler> ApplicationState<DB, AH> {
         }
     }
 
+    /// Commit changes from state store to application and persist changes to disk.
+    /// Returns application state hash.
+    ///
+    /// **Note**: changes from `check_tx` state is discarded and instead `deliver_tx` state used.
     pub fn commit(&mut self, multi_store: &mut ApplicationMultiBank<DB, AH::StoreKey>) -> [u8; 32] {
         self.check_mode.multi_store.tx_cache_clear();
         self.check_mode.multi_store.block_cache_clear();