Wrong preprocessor name in zmalloc.c

7 views
Skip to first unread message

Krzysztof Filipek

unread,
May 4, 2018, 9:21:04 AM5/4/18
to Redis DB
Hello,
I found in zmalloc.c:395 "HW_PYSMEM" instead of "HW_PHYSMEM". I have patch for this issue, so if I get good feedback I will create new pull request.

@@ -392,7 +392,7 @@ size_t zmalloc_get_memory_size(void) {
     mib
[0] = CTL_HW;
 
#if defined(HW_REALMEM)
     mib
[1] = HW_REALMEM;        /* FreeBSD. ----------------- */
-#elif defined(HW_PYSMEM)
+#elif defined(HW_PHYSMEM)
     mib
[1] = HW_PHYSMEM;        /* Others. ------------------ */
 
#endif
     
unsigned int size = 0;      /* 32-bit */




What do you think?

Best regards,
Krzysztof

Itamar Haber

unread,
May 4, 2018, 10:26:40 AM5/4/18
to Redis DB
Good catch - please PR

--
You received this message because you are subscribed to the Google Groups "Redis DB" group.
To unsubscribe from this group and stop receiving emails from it, send an email to redis-db+u...@googlegroups.com.
To post to this group, send email to redi...@googlegroups.com.
Visit this group at https://groups.google.com/group/redis-db.
For more options, visit https://groups.google.com/d/optout.
Reply all
Reply to author
Forward
0 new messages