[cfe-commits] small cleanup patch for libc++

Eitan Adler lists at eitanadler.com
Fri Dec 21 15:05:27 PST 2012


Its obvious from the context that there isn't a need for the double
initialization:

Index: src/locale.cpp
===================================================================
--- src/locale.cpp	(revision 170950)
+++ src/locale.cpp	(working copy)
@@ -2641,7 +2641,6 @@ utf16be_to_ucs4_length(const uint8_t* frm, const u
                        codecvt_mode mode = codecvt_mode(0))
 {
     const uint8_t* frm_nxt = frm;
-    frm_nxt = frm;
     if (mode & consume_header)
     {
         if (frm_end-frm_nxt >= 2 && frm_nxt[0] == 0xFE && frm_nxt[1] == 0xFF)
@@ -2774,7 +2773,6 @@ utf16le_to_ucs4_length(const uint8_t* frm, const u
                        codecvt_mode mode = codecvt_mode(0))
 {
     const uint8_t* frm_nxt = frm;
-    frm_nxt = frm;
     if (mode & consume_header)
     {
         if (frm_end-frm_nxt >= 2 && frm_nxt[0] == 0xFF && frm_nxt[1] == 0xFE)
@@ -2869,7 +2867,6 @@ utf16be_to_ucs2_length(const uint8_t* frm, const u
                        codecvt_mode mode = codecvt_mode(0))
 {
     const uint8_t* frm_nxt = frm;
-    frm_nxt = frm;
     if (mode & consume_header)
     {
         if (frm_end-frm_nxt >= 2 && frm_nxt[0] == 0xFE && frm_nxt[1] == 0xFF)


-- 
Eitan Adler



More information about the cfe-commits mailing list