MantisBT
Mantis Bug Tracker Workflow

View Revisions: Issue #32479 All Revisions ] Back to Issue ]
Summary 0032479: Application Framework - unnecessary API break within TDocStd_Application::Open()
Revision 2021-07-14 11:21 by kgv
Description TDocStd_Application::Open() definition has been changed to introduce new theFilter parameter:
  PCDM_ReaderStatus Open (Standard_IStream& theIStream,
      Handle(TDocStd_Document)& theDoc,
      const Handle(PCDM_ReaderFilter)& theFilter = Handle(PCDM_ReaderFilter)(),
      const Message_ProgressRange& theRange = Message_ProgressRange());


This API break doesn't look justified as most users will not use theFilter argument anyway.
It could be simply resolved by defining an additional method with fewer number of parameters or by moving theFilter to the end of argument list.
Revision 2021-07-14 11:21 by kgv
Description TDocStd_Application::Open() definition has been changed to introduce new theFilter parameter:
  PCDM_ReaderStatus Open (Standard_IStream& theIStream,
                          Handle(TDocStd_Document)& theDoc,
                          const Handle(PCDM_ReaderFilter)& theFilter = Handle(PCDM_ReaderFilter)(),
                          
const Message_ProgressRange& theRange = Message_ProgressRange());


This API break doesn't look justified as most users will not use theFilter argument anyway.
It could be simply resolved by defining an additional method with fewer number of parameters or by moving theFilter to the end of argument list.


Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker