Bug 1305422 - part 18 - remove #if 0'd-out code from nsStringIterator.h; r=erahm
authorNathan Froyd <froydnj@mozilla.com>
Thu, 29 Sep 2016 22:33:57 -0400
changeset 315889 c8c685a95ab025a21c418759f292bc491ab26957
parent 315888 f933f6215b9d67ab84255642d8ab2cb208f051c1
child 315890 37580f7e092f45a3ce05bca5e40ba4dfddacf221
push id20634
push usercbook@mozilla.com
push dateFri, 30 Sep 2016 10:10:13 +0000
treeherderfx-team@afe79b010d13 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerserahm
bugs1305422
milestone52.0a1
Bug 1305422 - part 18 - remove #if 0'd-out code from nsStringIterator.h; r=erahm We don't support IBM VisualAge for OS/2 anymore, and we haven't needed this code in a long time regardless.
xpcom/string/nsStringIterator.h
--- a/xpcom/string/nsStringIterator.h
+++ b/xpcom/string/nsStringIterator.h
@@ -51,25 +51,16 @@ public:
     return mPosition;
   }
 
   CharT operator*() const
   {
     return *get();
   }
 
-#if 0
-  // An iterator really deserves this, but some compilers (notably IBM VisualAge for OS/2)
-  //  don't like this when |CharT| is a type without members.
-  pointer operator->() const
-  {
-    return get();
-  }
-#endif
-
   self_type& operator++()
   {
     ++mPosition;
     return *this;
   }
 
   self_type operator++(int)
   {
@@ -163,26 +154,16 @@ public:
     return mPosition;
   }
 
   reference operator*() const
   {
     return *get();
   }
 
-#if 0
-  // An iterator really deserves this, but some compilers (notably IBM VisualAge for OS/2)
-  //  don't like this when |CharT| is a type without members.
-  pointer
-  operator->() const
-  {
-    return get();
-  }
-#endif
-
   self_type& operator++()
   {
     ++mPosition;
     return *this;
   }
 
   self_type operator++(int)
   {