Change file op |ssize_t|s to FileOperationResult

util/file/file_reader.h

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 19 matching lines @@
 class FileReaderInterface : public virtual FileSeekerInterface {
  public:
   virtual ~FileReaderInterface() {}
 
   //! \brief Wraps ReadFile(), or provides an implementation with identical
   //!     semantics.
   //!
   //! \return The number of bytes actually read if the operation succeeded,
   //!     which may be `0` or any positive value less than or equal to \a size.
   //!     `-1` if the operation failed, with an error message logged.
-  virtual ssize_t Read(void* data, size_t size) = 0;
+  virtual FileOperationResult Read(void* data, size_t size) = 0;
 
   //! \brief Wraps Read(), ensuring that the read succeeded and exactly \a size
   //!     bytes were read.
   //!
   //! Semantically, this behaves as LoggingReadFile().
   //!
   //! \return `true` if the operation succeeded, `false` if it failed, with an
   //!     error message logged. Short reads are treated as failures.
   bool ReadExactly(void* data, size_t size);
 };
@@ skipping 11 matching lines @@
 //! disk-based files.
 //!
 //! This class is expected to be used when other code is responsible for
 //! opening files and already provides file handles.
 class WeakFileHandleFileReader : public FileReaderInterface {
  public:
   explicit WeakFileHandleFileReader(FileHandle file_handle);
   ~WeakFileHandleFileReader() override;
 
   // FileReaderInterface:
-  ssize_t Read(void* data, size_t size) override;
+  FileOperationResult Read(void* data, size_t size) override;
 
   // FileSeekerInterface:
 
   //! \copydoc FileReaderInterface::Seek()
   //!
   //! \note This method is only guaranteed to function on file handles referring
   //!     to disk-based files.
   FileOffset Seek(FileOffset offset, int whence) override;
 
  private:
@@ skipping 35 matching lines @@
   //!     successful Open() that has not yet been matched by a subsequent call
   //!     to this method.
   void Close();
 
   // FileReaderInterface:
 
   //! \copydoc FileReaderInterface::Read()
   //!
   //! \note It is only valid to call this method between a successful Open() and
   //!     a Close().
-  ssize_t Read(void* data, size_t size) override;
+  FileOperationResult Read(void* data, size_t size) override;
 
   // FileSeekerInterface:
 
   //! \copydoc FileReaderInterface::Seek()
   //!
   //! \note It is only valid to call this method between a successful Open() and
   //!     a Close().
   FileOffset Seek(FileOffset offset, int whence) override;
 
  private:
@@ skipping 14 matching lines @@
 //!
 //! This class is expected to be used when other code is responsible for
 //! opening `FILE*` objects and already provides `FILE*` objects. A good use
 //! would be a WeakStdioFileReader for `stdin`.
 class WeakStdioFileReader : public FileReaderInterface {
  public:
   explicit WeakStdioFileReader(FILE* file);
   ~WeakStdioFileReader() override;
 
   // FileReaderInterface:
-  ssize_t Read(void* data, size_t size) override;
+  FileOperationResult Read(void* data, size_t size) override;
 
   // FileSeekerInterface:
 
   //! \copydoc FileReaderInterface::Seek()
   //!
   //! \note This method is only guaranteed to function on `FILE*` objects
   //!     referring to disk-based files.
   FileOffset Seek(FileOffset offset, int whence) override;
 
  private:
   FILE* file_;  // weak
 
   DISALLOW_COPY_AND_ASSIGN(WeakStdioFileReader);
 };
 
 }  // namespace crashpad
 
 #endif  // CRASHPAD_UTIL_FILE_FILE_READER_H_