Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None
    • Environment:
      Linux

      Description

      in file.cpp line 83
      //////////////
      std::string File::convertBackSlashes(const std::string& src) {
      std::string::size_type pos = src.find('
      ');
      if (pos == std::string::npos)

      { return src; }

      std::string mod(src);
      while(pos != std::string::npos)

      { mod[pos] = '/'; pos = src.find('\\'); }

      return mod;
      }
      //////////////
      resolution should be "pos = mod.find("
      ");"
      ////////////
      if the string has more than one "/" src.find return always the first "/" character occurence and the loop "become" infinite

        Activity

        Alban Seurat created issue -
        Alban Seurat made changes -
        Field Original Value New Value
        Affects Version/s 0.9.8 [ 10782 ]
        Priority Trivial [ 5 ] Minor [ 4 ]
        Alban Seurat made changes -
        Description in file.cpp line 83
        //////////////
        std::string File::convertBackSlashes(const std::string& src) {
          std::string::size_type pos = src.find('\\');
          if (pos == std::string::npos) {
            return src;
          }
          std::string mod(src);
          while(pos != std::string::npos) {
            mod[pos] = '/';
            pos = src.find('\\');
          }
          return mod;
        }
        //////////////
        resolution should be "pos = mod.find("\\");"

        in file.cpp line 83
        //////////////
        std::string File::convertBackSlashes(const std::string& src) {
          std::string::size_type pos = src.find('\\');
          if (pos == std::string::npos) {
            return src;
          }
          std::string mod(src);
          while(pos != std::string::npos) {
            mod[pos] = '/';
            pos = src.find('\\');
          }
          return mod;
        }
        //////////////
        resolution should be "pos = mod.find("\\");"
        ////////////
        if the string has more than one "/" src.find return always the first "/" character occurence and the loop "become" infinite
        Curt Arnold made changes -
        Fix Version/s 0.9.8 [ 10782 ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]

          People

          • Assignee:
            Curt Arnold
            Reporter:
            Alban Seurat
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development