 Chromium Code Reviews
 Chromium Code Reviews Issue 11593007:
  Replace the use CharacterStreams in Heap::AllocateSymbolInternal and String::ComputeHash  (Closed) 
  Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
    
  
    Issue 11593007:
  Replace the use CharacterStreams in Heap::AllocateSymbolInternal and String::ComputeHash  (Closed) 
  Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge| OLD | NEW | 
|---|---|
| 1 // Copyright 2012 the V8 project authors. All rights reserved. | 1 // Copyright 2012 the V8 project authors. All rights reserved. | 
| 2 // Redistribution and use in source and binary forms, with or without | 2 // Redistribution and use in source and binary forms, with or without | 
| 3 // modification, are permitted provided that the following conditions are | 3 // modification, are permitted provided that the following conditions are | 
| 4 // met: | 4 // met: | 
| 5 // | 5 // | 
| 6 // * Redistributions of source code must retain the above copyright | 6 // * Redistributions of source code must retain the above copyright | 
| 7 // notice, this list of conditions and the following disclaimer. | 7 // notice, this list of conditions and the following disclaimer. | 
| 8 // * Redistributions in binary form must reproduce the above | 8 // * Redistributions in binary form must reproduce the above | 
| 9 // copyright notice, this list of conditions and the following | 9 // copyright notice, this list of conditions and the following | 
| 10 // disclaimer in the documentation and/or other materials provided | 10 // disclaimer in the documentation and/or other materials provided | 
| (...skipping 746 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 757 uint32_t hash_field); | 757 uint32_t hash_field); | 
| 758 | 758 | 
| 759 MUST_USE_RESULT inline MaybeObject* AllocateAsciiSymbol( | 759 MUST_USE_RESULT inline MaybeObject* AllocateAsciiSymbol( | 
| 760 Vector<const char> str, | 760 Vector<const char> str, | 
| 761 uint32_t hash_field); | 761 uint32_t hash_field); | 
| 762 | 762 | 
| 763 MUST_USE_RESULT inline MaybeObject* AllocateTwoByteSymbol( | 763 MUST_USE_RESULT inline MaybeObject* AllocateTwoByteSymbol( | 
| 764 Vector<const uc16> str, | 764 Vector<const uc16> str, | 
| 765 uint32_t hash_field); | 765 uint32_t hash_field); | 
| 766 | 766 | 
| 767 template<typename T> | |
| 767 MUST_USE_RESULT MaybeObject* AllocateInternalSymbol( | 768 MUST_USE_RESULT MaybeObject* AllocateInternalSymbol( | 
| 768 unibrow::CharacterStream* buffer, int chars, uint32_t hash_field); | 769 T t, int chars, uint32_t hash_field); | 
| 769 | 770 | 
| 770 MUST_USE_RESULT MaybeObject* AllocateExternalSymbol( | 771 MUST_USE_RESULT MaybeObject* AllocateExternalSymbol( | 
| 
Yang
2012/12/17 14:30:34
Let's remove this function header, while we are at
 | |
| 771 Vector<const char> str, | 772 Vector<const char> str, | 
| 772 int chars); | 773 int chars); | 
| 773 | 774 | 
| 774 // Allocates and partially initializes a String. There are two String | 775 // Allocates and partially initializes a String. There are two String | 
| 775 // encodings: ASCII and two byte. These functions allocate a string of the | 776 // encodings: ASCII and two byte. These functions allocate a string of the | 
| 776 // given length and set its map and length fields. The characters of the | 777 // given length and set its map and length fields. The characters of the | 
| 777 // string are uninitialized. | 778 // string are uninitialized. | 
| 778 // Returns Failure::RetryAfterGC(requested_bytes, space) if the allocation | 779 // Returns Failure::RetryAfterGC(requested_bytes, space) if the allocation | 
| 779 // failed. | 780 // failed. | 
| 780 // Please note this does not perform a garbage collection. | 781 // Please note this does not perform a garbage collection. | 
| (...skipping 2098 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 2879 AssertNoAllocation no_alloc; // i.e. no gc allowed. | 2880 AssertNoAllocation no_alloc; // i.e. no gc allowed. | 
| 2880 | 2881 | 
| 2881 private: | 2882 private: | 
| 2882 DISALLOW_IMPLICIT_CONSTRUCTORS(PathTracer); | 2883 DISALLOW_IMPLICIT_CONSTRUCTORS(PathTracer); | 
| 2883 }; | 2884 }; | 
| 2884 #endif // DEBUG || LIVE_OBJECT_LIST | 2885 #endif // DEBUG || LIVE_OBJECT_LIST | 
| 2885 | 2886 | 
| 2886 } } // namespace v8::internal | 2887 } } // namespace v8::internal | 
| 2887 | 2888 | 
| 2888 #endif // V8_HEAP_H_ | 2889 #endif // V8_HEAP_H_ | 
| OLD | NEW |