YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Optimize block size calculation in functions working with linked list.

Browse Source 
JerryScript-DCO-1.0-Signed-off-by: Andrey Shitov a.shitov@samsung.com
This commit is contained in:
Andrey Shitov
2015-07-30 20:07:50 +03:00
parent af56cd8465
commit 24e307481e
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/parser/js/collections/linked-list.cpp
+6 -5
View File
@@ -29,15 +29,16 @@ do { \
JERRY_ASSERT (header); \ JERRY_ASSERT (header); \
} while (0); } while (0);
static size_t linked_list_block_size (uint16_t element_size) static size_t linked_list_block_size ()
{ {
return jsp_mm_recommend_size (sizeof (linked_list_header) + element_size) - sizeof (linked_list_header); return jsp_mm_recommend_size (sizeof (linked_list_header) + 1) - sizeof (linked_list_header);
} }
linked_list linked_list
linked_list_init (uint16_t element_size) linked_list_init (uint16_t element_size)
{ {
size_t size = sizeof (linked_list_header) + linked_list_block_size (element_size); JERRY_ASSERT (element_size <= linked_list_block_size ());
size_t size = sizeof (linked_list_header) + linked_list_block_size ();
linked_list list = (linked_list) jsp_mm_alloc (size); linked_list list = (linked_list) jsp_mm_alloc (size);
if (list == null_list) if (list == null_list)
{ {
@@ -68,7 +69,7 @@ linked_list_element (linked_list list, size_t element_num)
{ {
ASSERT_LIST (list); ASSERT_LIST (list);
linked_list_header *header = (linked_list_header *) list; linked_list_header *header = (linked_list_header *) list;
size_t block_size = linked_list_block_size (header->element_size); size_t block_size = linked_list_block_size ();
linked_list raw = list + sizeof (linked_list_header); linked_list raw = list + sizeof (linked_list_header);
if (block_size < header->element_size * (element_num + 1)) if (block_size < header->element_size * (element_num + 1))
{ {
@@ -91,7 +92,7 @@ linked_list_set_element (linked_list list, size_t element_num, void *element)
{ {
ASSERT_LIST (list); ASSERT_LIST (list);
linked_list_header *header = (linked_list_header *) list; linked_list_header *header = (linked_list_header *) list;
size_t block_size = linked_list_block_size (header->element_size); size_t block_size = linked_list_block_size ();
uint8_t *raw = (uint8_t *) (header + 1); uint8_t *raw = (uint8_t *) (header + 1);
if (block_size < header->element_size * (element_num + 1)) if (block_size < header->element_size * (element_num + 1))
{ {