FileAccess: Introduce advance() for skipping bytes #106308
                
     Open
            
            
          
      
        
          +13
        
        
          −0
        
        
          
        
      
    
  
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
This adds a
advance(offset)method to FileAccess, providing a more readable & streamlined way to move the file cursor forward/backward. Unless I'm missing something, the only way to do this is by manually callingseek(file.get_position() + amount), which feels unnecessarily verbose for an operation like this. While you can also use methods likeget_32()orget_buffer()to advance the cursor, these actually read data into memory, which is something that can negatively impact performance when skipping over large segments (it's also not very explicit that these functions do that). This method offers a more readable and purpose-specific alternative.