Apply suggestions from code review

Co-authored-by: Jonas Rembser <jonas.rembser@cern.ch>

Author: ferdymercury

graf2d/cocoa/inc/TGQuartz.h

@@ -96,7 +96,7 @@ class TGQuartz : public TGCocoa {
    void AlignTTFString();
    Bool_t IsTTFStringVisible(Int_t x, Int_t y, UInt_t w, UInt_t h);
    void RenderTTFString(Int_t x, Int_t y, ETextMode mode);
-   //TODO: use void * instead of QuartzPixmap *
+   //TODO: move internal headers like TGQuartz.h out of the public ROOT user interface. Then we can also use XQuartz types like QuartzPixmap* instead of void * in functions like these
    void DrawFTGlyphIntoPixmap(void *pixmap, FT_Bitmap *source, ULong_t fore, ULong_t back, Int_t bx, Int_t by);
 
    void SetAA();

graf2d/fitsio/src/TFITS.cxx

@@ -42,7 +42,7 @@
 ///
 /// Please have a look to the tutorials ($ROOTSYS/tutorials/fitsio/) to see
 /// some examples. IMPORTANT: to run tutorials it is required that
-/// you change the current working directory of ROOT (Cling) shell to the
+/// you change the current working directory of the ROOT shell to the
 /// tutorials directory. Example:
 /// ~~~ {.cpp}
 /// root [1] gSystem->ChangeDirectory("tutorials/fitsio")

graf2d/win32gdk/inc/TGWin32ProxyDefs.h

@@ -13,7 +13,7 @@
 #ifndef ROOT_TGWin32ProxyDefs
 #define ROOT_TGWin32ProxyDefs
 
-#include "Rtypes.h"  // Cling headers
+#include "Rtypes.h"
 
 
 #define _NAME4_(name1,name2,name3,name4) _NAME3_(name1,name2,name3)name4

graf3d/eve/src/TEveBrowser.cxx

@@ -463,7 +463,7 @@ Specialization of TRootBrowser for Eve.
 
 
 ////////////////////////////////////////////////////////////////////////////////
-/// Add "Export to CINT" (now Cling) into context-menu for class cl.
+/// Add "Export to CINT" into context-menu for class cl.
 
 void TEveBrowser::SetupCintExport(TClass* cl)
 {

graf3d/eve/src/TEveElement.cxx

@@ -512,7 +512,7 @@ void TEveElement::SaveVizParams(std::ostream& out, const TString& tag, const TSt
 /// Write-out visual parameters for this object.
 /// This is a virtual function and all sub-classes are required to
 /// first call the base-element version.
-/// The name of the element pointer is 'x%03d', due to historial CINT limitations.
+/// The name of the element pointer is 'x%03d', due to historical CINT limitations.
 /// Three spaces should be used for indentation, same as in
 /// SavePrimitive() methods.
 

graf3d/gl/inc/TGLParametric.h

@@ -49,7 +49,6 @@ class TGLParametricEquation : public TNamed {
    Bool_t               fModified;
 
    typedef std::unique_ptr<TGLHistPainter> Painter_t;
-   //C++ compiler do not need TGLhHstPainter definition here, but I'm not sure about Cling,
    //so I've included TGLHistPainter definition.
    Painter_t            fPainter;
 

gui/ged/src/TStyleManager.cxx

@@ -78,7 +78,7 @@ style is removed from the list of all available styles for the
 current ROOT session. WARNING: it will be lost if you didn't saved
 it in a C++ macro file before its deletion. Also, you cannot delete
 the selected style if it is set to gStyle. A message 'Can not delete
-gStyle' will be displayed on the Cling prompt.
+gStyle' will be displayed on the ROOT prompt.
 
 Editor's buttons:
 Open / close the style editor:

io/io/src/TStreamerInfo.cxx

@@ -1108,7 +1108,7 @@ void TStreamerInfo::BuildCheck(TFile *file /* = 0 */, Bool_t load /* = kTRUE */)
                // match in the 'current' in memory TStreamerInfo for
                // a non foreign class (we can not get here if this is
                // a foreign class so we do not need to test it),
-               // we need to add this one more test since the Cling behaviour
+               // we need to add this one more test since the interpreter behaviour
                // with enums changed over time, so verify the checksum ignoring
                // members of type enum. We also used to not count the //[xyz] comment
                // in the checksum, so test for that too.
@@ -1158,7 +1158,7 @@ void TStreamerInfo::BuildCheck(TFile *file /* = 0 */, Bool_t load /* = kTRUE */)
                // match in the 'current' in memory TStreamerInfo for
                // a non foreign class (we can not get here if this is
                // a foreign class so we do not need to test it),
-               // we need to add this one more test since the Cling behaviour
+               // we need to add this one more test since the interpreter behaviour
                // with enums changed over time, so verify the checksum ignoring
                // members of type enum. We also used to not count the //[xyz] comment
                // in the checksum, so test for that too.

io/xml/src/TXMLFile.cxx

@@ -24,7 +24,7 @@
 // The XML file is built in memory before being dumped to disk.
 //
 // Like for normal ROOT files, XML files use the same I/O mechanism
-// exploiting the ROOT/Cling dictionary. Any class having a dictionary
+// exploiting the I/O dictionary. Any class having a dictionary
 // can be saved in XML format.
 //
 // This first implementation does not support subdirectories

tree/treeplayer/inc/TBranchProxyDirector.h

@@ -47,7 +47,7 @@ namespace Internal{
    public:
 
       TBranchProxyDirector(TTree* tree, Long64_t i);
-      TBranchProxyDirector(TTree* tree, Int_t i);     // Historial: CINT had a problem casting int to long long
+      TBranchProxyDirector(TTree* tree, Int_t i);     // Historical: CINT had a problem casting int to long long
 
       void     Attach(Detail::TBranchProxy* p);
       void     Attach(TFriendProxy* f);