Hygine cleanups in holders.h

Use "= delete" syntax to make it clear the IcuHolder copy constructor
and assignment opperators are removed.

Remove superfluous "public" modifier on the struct closers used by the
IcuHolders.

diff --git a/src/corefx/System.Globalization.Native/holders.h b/src/corefx/System.Globalization.Native/holders.h
index 3e08189..9176de3 100644 (file)
--- a/src/corefx/System.Globalization.Native/holders.h
+++ b/src/corefx/System.Globalization.Native/holders.h
@@ -34,13 +34,12 @@ class IcuHolder
 
   private:
     T* m_p;
-    IcuHolder(const IcuHolder&);
-    IcuHolder operator=(const IcuHolder&);
+    IcuHolder(const IcuHolder&) = delete;
+    IcuHolder operator=(const IcuHolder&) = delete;
 };
 
 struct UCalendarCloser
 {
-  public:
     void operator()(UCalendar* pCal) const
     {
         ucal_close(pCal);
@@ -49,7 +48,6 @@ struct UCalendarCloser
 
 struct UEnumerationCloser
 {
-  public:
     void operator()(UEnumeration* pEnum) const
     {
         uenum_close(pEnum);
@@ -58,7 +56,6 @@ struct UEnumerationCloser
 
 struct UDateTimePatternGeneratorCloser
 {
-  public:
     void operator()(UDateTimePatternGenerator* pGenerator) const
     {
         udatpg_close(pGenerator);
@@ -67,7 +64,6 @@ struct UDateTimePatternGeneratorCloser
 
 struct UDateFormatCloser
 {
-  public:
     void operator()(UDateFormat* pDateFormat) const
     {
         udat_close(pDateFormat);