tools/rbuild/backend/mingw/modulehandler.h

[Reuben Perelman]

I was getting this error when trying to compile reactos:
>   reub2000@reub2000 ~/reactos $ make
>  [MKDIR] obj-i386/tools
>  [MKDIR] obj-i386/tools/rbuild
>  [MKDIR] obj-i386/tools/rbuild/backend
>  [MKDIR] obj-i386/tools/rbuild/backend/mingw
>  [CC] tools/rbuild/backend/mingw/mingw.cpp
>  tools/rbuild/backend/mingw/modulehandler.h:91: error: extra
>  qualification ‘MingwModuleHandler::’ on member
>  ‘GetCompilationUnitDependencies’
>  tools/rbuild/backend/mingw/modulehandler.h:150: error: extra
>  qualification ‘MingwModuleHandler::’ on member
>  ‘GetPrecompiledHeaderFilename’
>  make: *** [obj-i386/tools/rbuild/backend/mingw/mingw.o] Error 1 
Looking at the file, I couldn't understand why MingwModuleHandler:: was
being used within the class of the same name, so I removed it from
before the definition of the functions. I'm guessing that it was copied
from the function definition outside of the class in a .cpp file?
Anyways, this seems to allow reactos to compile:
>   Index: tools/rbuild/backend/mingw/modulehandler.h
>  ===================================================================
>  --- tools/rbuild/backend/mingw/modulehandler.h (revision 20484)
>  +++ tools/rbuild/backend/mingw/modulehandler.h (working copy)
>  @@ -88,7 +88,7 @@
>  std::string GetBasename ( const std::string& filename ) const;
>  FileLocation* GetActualSourceFilename ( const FileLocation*
>  fileLocation ) const;
>  std::string GetExtraDependencies ( const std::string& filename ) const;
>  - std::string MingwModuleHandler::GetCompilationUnitDependencies (
>  const CompilationUnit& compilationUnit ) const;
>  + std::string GetCompilationUnitDependencies ( const CompilationUnit&
>  compilationUnit ) const;
>  std::string GetModuleArchiveFilename () const;
>  bool IsGeneratedFile ( const File& file ) const;
>  std::string GetImportLibraryDependency ( const Module& importedModule );
>  @@ -147,7 +147,7 @@
>  std::string GenerateGccIncludeParameters () const;
>  std::string GenerateGccParameters () const;
>  std::string GenerateNasmParameters () const;
>  - std::string MingwModuleHandler::GetPrecompiledHeaderFilename () const;
>  + std::string GetPrecompiledHeaderFilename () const;
>  void GenerateGccCommand ( const FileLocation* sourceFileLocation,
>  const std::string& extraDependencies,
>  const std::string& cc, 
Also, I saw a lot of 'std::'. Why not use 'using namespace std;'?

[Royce Mitchell III]

Reuben Perelman wrote:
>   Looking at the file, I couldn't understand why
> MingwModuleHandler::
>  was being used within the class of the same name, so I removed it from
>  before the definition of the functions. I'm guessing that it was
>  copied from the function definition outside of the class in a .cpp file?
>  
Good catch, sorry about that.
>   Also, I saw a lot of 'std::'. Why not use
> 'using namespace std;'? 
It's {very} bad practice to put using statements in header files -
almost as bad as using goto :P

[Reuben Perelman]

Royce Mitchell III wrote:
>   Reuben Perelman wrote:
> >   Looking at the file, I couldn't understand
> > why MingwModuleHandler::
> >  was being used within the class of the same name, so I removed it
> >  from before the definition of the functions. I'm guessing that it was
> >  copied from the function definition outside of the class in a .cpp file?
> >  
>  Good catch, sorry about that. 
Somebody with write access to svn should fix it.
> >   Also, I saw a lot of 'std::'. Why not use
> > 'using namespace std;'? 
>  It's {very} bad practice to put using statements in header files -
>  almost as bad as using goto :P
>  _______________________________________________
>  Ros-dev mailing list
>  Ros-dev(a)reactos.org
>  http://www.reactos.org/mailman/listinfo/ros-dev
>

[Reuben Perelman]

Somebody with write access should fix this simple little issue.
I also filed this bug on it:
http://www.reactos.org/bugzilla/show_bug.cgi?id=1240
Reuben Perelman wrote:
>   Royce Mitchell III wrote:
> >   Reuben Perelman wrote:
> > >   Looking at the file, I couldn't understand
> > > why MingwModuleHandler::
> > >  was being used within the class of the same name, so I removed it
> > >  from before the definition of the functions. I'm guessing that it
> > >  was copied from the function definition outside of the class in a
> > >  .cpp file?
> > >  
> >  Good catch, sorry about that. 
>  Somebody with write access to svn should fix it.
> > >   Also, I saw a lot of 'std::'. Why not use
> > > 'using namespace std;'? 
> >  It's {very} bad practice to put using statements in header files -
> >  almost as bad as using goto :P
> >  _______________________________________________
> >  Ros-dev mailing list
> >  Ros-dev(a)reactos.org
> >  http://www.reactos.org/mailman/listinfo/ros-dev
> >  
>  _______________________________________________
>  Ros-dev mailing list
>  Ros-dev(a)reactos.org
>  http://www.reactos.org/mailman/listinfo/ros-dev
>

[Reuben Perelman]

Now a similar problem exists in tools/cabman/cabinet.h.
This fixed it:
Index: tools/cabman/cabinet.h
===================================================================
--- tools/cabman/cabinet.h      (revision 20506)
+++ tools/cabman/cabinet.h      (working copy)
@@ -309,9 +309,9 @@
     /* Default destructor */
     virtual ~CCabinet();
     /* Determines if a character is a separator */
-    bool CCabinet::IsSeparator(char Char);
+    bool IsSeparator(char Char);
     /* Replaces \ or / with the one used be the host environment */
-    char* CCabinet::ConvertPath(char* Path, bool Allocate);
+    char* ConvertPath(char* Path, bool Allocate);
     /* Returns a pointer to the filename part of a fully qualified
filename */
     char* GetFileName(char* Path);
     /* Removes a filename from a fully qualified filename */
Reuben Perelman wrote:
>   I was getting this error when trying to compile
> reactos:
> >   reub2000@reub2000 ~/reactos $ make
> >  [MKDIR] obj-i386/tools
> >  [MKDIR] obj-i386/tools/rbuild
> >  [MKDIR] obj-i386/tools/rbuild/backend
> >  [MKDIR] obj-i386/tools/rbuild/backend/mingw
> >  [CC] tools/rbuild/backend/mingw/mingw.cpp
> >  tools/rbuild/backend/mingw/modulehandler.h:91: error: extra
> >  qualification ‘MingwModuleHandler::’ on member
> >  ‘GetCompilationUnitDependencies’
> >  tools/rbuild/backend/mingw/modulehandler.h:150: error: extra
> >  qualification ‘MingwModuleHandler::’ on member
> >  ‘GetPrecompiledHeaderFilename’
> >  make: *** [obj-i386/tools/rbuild/backend/mingw/mingw.o] Error 1 
>  Looking at the file, I couldn't understand why MingwModuleHandler::
>  was being used within the class of the same name, so I removed it from
>  before the definition of the functions. I'm guessing that it was
>  copied from the function definition outside of the class in a .cpp file?
>  Anyways, this seems to allow reactos to compile:
> >   Index: tools/rbuild/backend/mingw/modulehandler.h
> >  ===================================================================
> >  --- tools/rbuild/backend/mingw/modulehandler.h (revision 20484)
> >  +++ tools/rbuild/backend/mingw/modulehandler.h (working copy)
> >  @@ -88,7 +88,7 @@
> >  std::string GetBasename ( const std::string& filename ) const;
> >  FileLocation* GetActualSourceFilename ( const FileLocation*
> >  fileLocation ) const;
> >  std::string GetExtraDependencies ( const std::string& filename ) const;
> >  - std::string MingwModuleHandler::GetCompilationUnitDependencies (
> >  const CompilationUnit& compilationUnit ) const;
> >  + std::string GetCompilationUnitDependencies ( const CompilationUnit&
> >  compilationUnit ) const;
> >  std::string GetModuleArchiveFilename () const;
> >  bool IsGeneratedFile ( const File& file ) const;
> >  std::string GetImportLibraryDependency ( const Module& importedModule );
> >  @@ -147,7 +147,7 @@
> >  std::string GenerateGccIncludeParameters () const;
> >  std::string GenerateGccParameters () const;
> >  std::string GenerateNasmParameters () const;
> >  - std::string MingwModuleHandler::GetPrecompiledHeaderFilename () const;
> >  + std::string GetPrecompiledHeaderFilename () const;
> >  void GenerateGccCommand ( const FileLocation* sourceFileLocation,
> >  const std::string& extraDependencies,
> >  const std::string& cc, 
>  Also, I saw a lot of 'std::'. Why not use 'using namespace std;'?
>  _______________________________________________
>  Ros-dev mailing list
>  Ros-dev(a)reactos.org
>  http://www.reactos.org/mailman/listinfo/ros-dev
>