Repository: xbmc/xbmc
File: xbmc/filesystem/IDirectory.h (C/C++)

Code before:
  \brief Create the directory
  \param url Directory to create.
  \return Returns \e true, if directory is created or if it already exists
  \sa CDirectoryFactory
  */
  virtual bool Create(const CURL& url) { return false; }
  /*!
  \brief Check for directory existence
  \param url Directory to check.
  \return Returns \e true, if directory exists
  \sa CDirectoryFactory
  */
  virtual bool Exists(const CURL& url) { return false; }
  /*!
  \brief Removes the directory
  \param url Directory to remove.
  \return Returns \e false if not successful
  */
  virtual bool Remove(const CURL& url) { return false; }

  /*!
  \brief Provided a path, attempts to resolve to a mount point
  \param path Path to resolve
  \return Returns the mountpoint if found, else the provided path
  */
  virtual std::string ResolveMountPoint(const std::string& path) { return path; }

  /*!
  \brief Recursively removes the directory
  \param url Directory to remove.
  \return Returns \e false if not successful
  */
  virtual bool RemoveRecursive(const CURL& url) { return false; }

  /*!
  \brief Whether this file should be listed
  \param url File to test.
  \return Returns \e true if the file should be listed
  */
  virtual bool IsAllowed(const CURL& url) const;

  /*! \brief Whether to allow all files/folders to be listed.
   \return Returns \e true if all files/folder should be listed.
   */
  virtual bool AllowAll() const { return false; }

  /*!
  \brief How this directory should be cached
  \param url Directory at hand.
  \return Returns the cache type.
  */

Review comment (suggestion):
```suggestion
  virtual std::string ResolveMountPoint(const std::string& path) const { return path; }
```

Code after:
  \brief Create the directory
  \param url Directory to create.
  \return Returns \e true, if directory is created or if it already exists
  \sa CDirectoryFactory
  */
  virtual bool Create(const CURL& url) { return false; }
  /*!
  \brief Check for directory existence
  \param url Directory to check.
  \return Returns \e true, if directory exists
  \sa CDirectoryFactory
  */
  virtual bool Exists(const CURL& url) { return false; }
  /*!
  \brief Removes the directory
  \param url Directory to remove.
  \return Returns \e false if not successful
  */
  virtual bool Remove(const CURL& url) { return false; }

  /*!
  \brief Provided a path, attempts to resolve to a mount point
  \param path Path to resolve
  \return Returns the mountpoint if found, else the provided path
  */
  virtual std::string ResolveMountPoint(const std::string& path) const { return path; }

  /*!
  \brief Recursively removes the directory
  \param url Directory to remove.
  \return Returns \e false if not successful
  */
  virtual bool RemoveRecursive(const CURL& url) { return false; }

  /*!
  \brief Whether this file should be listed
  \param url File to test.
  \return Returns \e true if the file should be listed
  */
  virtual bool IsAllowed(const CURL& url) const;

  /*! \brief Whether to allow all files/folders to be listed.
   \return Returns \e true if all files/folder should be listed.
   */
  virtual bool AllowAll() const { return false; }

  /*!
  \brief How this directory should be cached
  \param url Directory at hand.
  \return Returns the cache type.
  */