Browse Source

Minor unrelated cleanup

rhuanjl 5 years ago
parent
commit
f6fc3ade72
2 changed files with 2 additions and 5 deletions
  1. 2 1
      lib/Common/Common/NumberUtilities_strtod.cpp
  2. 0 4
      test/runtests.py

+ 2 - 1
lib/Common/Common/NumberUtilities_strtod.cpp

@@ -1599,7 +1599,8 @@ static BOOL FDblToRgbPrecise(double dbl, __out_ecount(kcbMaxRgb) byte *prgb, int
                 // Do not always push to higherBound
                 // See Js::NumberUtilities::FDblToStr for the exception
                 // i.e. we shouldn't push digits beyond interest to higherBound
-                prgb[ib++] = bT + (byte)(nDigits == -1 || ib < nDigits ? 1 : 0);
+                prgb[ib] = bT + (byte)(nDigits == -1 || ib < nDigits ? 1 : 0);
+                ++ib;
                 break;
             }
 LRoundUp9:

+ 0 - 4
test/runtests.py

@@ -119,13 +119,11 @@ if sys.platform != 'win32':
 arch_alias = 'amd64' if arch == 'x64' else None
 
 # flavor: debug, test, release
-type_flavor = {'chk':'Debug', 'test':'Test', 'fre':'Release'}
 flavor = 'Debug' if args.debug else ('Test' if args.test else None)
 if flavor == None:
     print("ERROR: Test build target wasn't defined.")
     print("Try '-t' (test build) or '-d' (debug build).")
     sys.exit(1)
-flavor_alias = 'chk' if flavor == 'Debug' else 'fre'
 
 # handling for extra flags
 extra_flags = ['-WERExceptionSupport']
@@ -160,8 +158,6 @@ not_tags = set(args.not_tag or []).union(['fail', 'exclude_' + arch, 'exclude_'
 
 if arch_alias:
     not_tags.add('exclude_' + arch_alias)
-if flavor_alias:
-    not_tags.add('exclude_' + flavor_alias)
 if args.only_slow:
     tags.add('Slow')
 elif not args.include_slow: