remove TODOs

equilibria-xyz · Jul 21, 2023 · fc47d49 · fc47d49
1 parent 115f1a9
commit fc47d49
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/perennial/contracts/Market.sol b/packages/perennial/contracts/Market.sol
@@ -315,7 +315,7 @@ contract Market is IMarket, Instance {
 
         // oracle
         (context.latestVersion, context.currentTimestamp) = _oracleVersion();
-        context.positionVersion = _oracleVersionAtPosition(context, _position.read()); // TODO: remove this
+        context.positionVersion = _oracleVersionAtPosition(context, _position.read());
 
         // after
         _endGas(context);
@@ -408,15 +408,15 @@ contract Market is IMarket, Instance {
     /// @param newPosition The pending position to process
     function _processPositionGlobal(Context memory context, Position memory newPosition) private {
         Version memory version = _versions[context.latestPosition.global.timestamp].read();
-        OracleVersion memory oracleVersion = _oracleVersionAtPosition(context, newPosition); // TODO: seems weird some logic is in here
+        OracleVersion memory oracleVersion = _oracleVersionAtPosition(context, newPosition);
         if (!oracleVersion.valid) newPosition.invalidate(context.latestPosition.global);
 
         (uint256 fromTimestamp, uint256 fromId) = (context.latestPosition.global.timestamp, context.latestPosition.global.id);
         (VersionAccumulationResult memory accumulationResult, UFixed6 accumulatedFee) = version.accumulate(
             context.global,
             context.latestPosition.global,
             newPosition,
-            context.positionVersion, // TODO: ??
+            context.positionVersion,
             oracleVersion,
             context.marketParameter,
             context.riskParameter