Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-15809

Replace boost::regex in utils package

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: utils
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Arch 2018-09-24, Arch 2018-10-01
    • Team:
      Architecture

      Description

      The utils package uses boost::regex for sexagesimal string parsing and for demangling. Neither of these examples need the power of boost::regex and std::regex can be used instead.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          No builds found.
          tjenness Tim Jenness created issue -
          tjenness Tim Jenness made changes -
          Field Original Value New Value
          Risk Score 0
          tjenness Tim Jenness made changes -
          Sprint Arch 2018-09-24 [ 784 ]
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Kian-Tat Lim Changes were pretty simple to switch to std::regex. All tests pass. I'm starting a Jenkins run.

          Show
          tjenness Tim Jenness added a comment - Kian-Tat Lim Changes were pretty simple to switch to std::regex. All tests pass. I'm starting a Jenkins run.
          tjenness Tim Jenness made changes -
          Reviewers Kian-Tat Lim [ ktl ]
          Status To Do [ 10001 ] In Review [ 10004 ]
          tjenness Tim Jenness made changes -
          Assignee Tim Jenness [ tjenness ]
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          Some minor suggestions.

          Show
          ktl Kian-Tat Lim added a comment - Some minor suggestions.
          ktl Kian-Tat Lim made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          ktl Kian-Tat Lim made changes -
          Sprint Arch 2018-09-24 [ 784 ] Arch 2018-09-24, Arch 2018-10-01 [ 784, 788 ]
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Merged much simpler form since DM-15831 removed most of the regex code.

          Show
          tjenness Tim Jenness added a comment - Merged much simpler form since DM-15831 removed most of the regex code.
          tjenness Tim Jenness made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            tjenness Tim Jenness
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            Kian-Tat Lim
            Watchers:
            Kian-Tat Lim, Russell Owen, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.