improve some natspecs on BasicSwap7683 contract

Author: lmcorbalan

solidity/src/BasicSwap7683.sol

@@ -85,6 +85,9 @@ abstract contract BasicSwap7683 is Base7683 {
 
     /**
      * @dev Settles multiple orders by dispatching the settlement instructions.
+     * The proper status of all the orders (filled) is validated on the Base7683 before calling this function.
+     * It assumes that all orders were originated in the same originDomain so it uses the the one from the first one for
+     * dispatching the message, but if some order differs on the originDomain it can be re-settle later.
      * @param _orderIds The IDs of the orders to settle.
      * @param _ordersOriginData The original data of the orders.
      * @param _ordersFillerData The filler data for the orders.
@@ -104,23 +107,27 @@ abstract contract BasicSwap7683 is Base7683 {
 
     /**
      * @dev Refunds multiple OnchainCrossChain orders by dispatching refund instructions.
+     * The proper status of all the orders (NOT filled and expired) is validated on the Base7683 before calling this
+     * function.
+     * It assumes that all orders were originated in the same originDomain so it uses the the one from the first one for
+     * dispatching the message, but if some order differs on the originDomain it can be re-refunded later.
      * @param _orders The orders to refund.
      * @param _orderIds The IDs of the orders to refund.
      */
     function _refundOrders(OnchainCrossChainOrder[] memory _orders, bytes32[] memory _orderIds) internal override {
-        // at this point we are sure all orders are NOT filled, use the first order to get the originDomain
-        // if some order differs on the originDomain it can be re-refunded later
         _dispatchRefund(OrderEncoder.decode(_orders[0].orderData).originDomain, _orderIds);
     }
 
     /**
      * @dev Refunds multiple GaslessCrossChain orders by dispatching refund instructions.
+     * The proper status of all the orders (NOT filled and expired) is validated on the Base7683 before calling this
+     * function.
+     * It assumes that all orders were originated in the same originDomain so it uses the the one from the first one for
+     * dispatching the message, but if some order differs on the originDomain it can be re-refunded later.
      * @param _orders The orders to refund.
      * @param _orderIds The IDs of the orders to refund.
      */
     function _refundOrders(GaslessCrossChainOrder[] memory _orders, bytes32[] memory _orderIds) internal override {
-        // at this point we are sure all orders are NOT filled, use the first order to get the originDomain
-        // if some order differs on the originDomain it can be re-refunded later
         _dispatchRefund(OrderEncoder.decode(_orders[0].orderData).originDomain, _orderIds);
     }