Uploaded image for project: 'SystemDS'
  1. SystemDS
  2. SYSTEMDS-1477

Create ParserFactory

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • SystemML 0.14
    • Parser
    • None

    Description

      Code such as the following does not really follow a recognized design pattern:

      AParserWrapper parser = AParserWrapper.createParser(false);
      

      For ease of code maintenance, this should be replaced by something such as:

      ParserWrapper parser = ParserFactory.createParser(false);
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            deron Jon Deron Eriksson
            deron Jon Deron Eriksson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment