Говнокод

if ((NULL != (string_array = (0 == *length) ? malloc((*length + 1) * REG_SIZE) : realloc(string_array, (*length + 1) * REG_SIZE))))

А кому-то это говно меинтейнить надо будет...

Запостил:

codemonkey, 08.10.2014 (Updated 28.03.2018)

Комментарии (9) RSS

Abbath 12 лет ago # 0

Код понятный.
Ответить
- bormand 12 лет ago # 0 ⇈
  Ага. Но почему-то все забывают, что realloc способен работать как malloc (да и как free):
```
string_array = realloc(string_array, (*length + 1) * REG_SIZE);
if (string_array) {
    // ...
}
```
  Ответить
  - bormand 12 лет ago # 0 ⇈
    
    P.S. Утечка памяти при out of memory сохранена 😉
    Ответить
    - guest 12 лет ago # 0 ⇈
      
      А разве сейчас что-то сишное расчитано на корректную работу при Out Of Memory?
      Ответить
      - kipar 12 лет ago # 0 ⇈
        
        >А разве вообще что-то расчитано на корректную работу при Out Of Memory?
        fixed
        Ответить
        
        inkanus-gray 12 лет ago # 0 ⇈
        
        Redis вроде умеет удалять записи, которые он посчитает «лишними», при возникновении Out Of Memory.
        Ответить
        
        bormand 12 лет ago # 0 ⇈
        
        LRU? При использовании в качестве кеша в оперативке - вполне логичный ход.
        Ответить
  - Abbath 12 лет ago # 0 ⇈
    
    Документацию никто не читает. Скопипастил пример и готово.
    "Хуяк, хуяк и в продакшн" как говорится.
    Ответить
  - roman-kashitsyn 12 лет ago # 0 ⇈
    > что realloc способен работать как malloc (да и как free)
    Кстати, эта фича realloc используется в сишном интерфейсе lua. Там "интерфейс" аллокатора спроектирован под влияним realloc
    http://www.lua.org/manual/5.1/manual.html#lua%5FAlloc
    Ответить