- 1
- 2
- 3
#define _XYNTA_SKIP \
continue; \
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 48
−45
#define _XYNTA_SKIP \
continue; \
}
Зачѣмъ?
−52
if ((err = some_function(args)) != GREAT_SUCCESS) {
debug_print("some function failed, err = %u", err);
goto out;
}
out:
/* cleanup */
return err;
"ко-ко-ко оставить гото на future fucking use куд-кудах"
−106
int stupid(int a)
{
return a + 1 > a;
}
int main(void)
{
int a = -1;
printf("%d\n", stupid(a));
return 0;
}
И вот так один UB портит всё.
~ $ gcc -c test.c -O3
~ $ ./test
1
Disassembly of section .text:
0000000000000000 <stupid>:
0: b8 01 00 00 00 mov $0x1,%eax
5: c3 retq
−47
/* optimized division by 16 */
n >>= 4
/* optimized remainder by 16 */
k &= 0xF
И такой prematurely optimized хуеты в кодобазе ОЧЕНЬ МНОГО.
+140
/*
* To the unenlightened: This sets the X (where X is variable between calls) LSB's of mask to 1.
*/
mask = ~(~0 << X);
Байтопроблемушки. Me gusta.
+137
printk(KERN_CRIT "AAA\n");
Ебу и патчу.
+142
URxvt*foreground: #C0C0C0
Вот что монокай с urxvt делает.
+143
static int card_open(struct inode *inode, struct file *f)
{
unsigned int minor = MINOR(inode->i_rdev);
DEBUG(KERN_CRIT "Driver: card_open()\n");
f->private_data = (void *)(unsigned long long)minor;
return 0;
}
А вместо минора надобно захуярить указатель на struct some_idiot_wrote_this *asshole в f->private_data.
+142
fstat(fd, &s);
count = s.st_size % sizeof(uint32_t) == 0 ? s.st_size : s.st_size + (sizeof(uint32_t) - s.st_size % sizeof(uint32_t));
buf = malloc(count);
/* etc */
Собственно, round up to nearest 32-bit word. Нет ли метода покрасивше?
+142
if (false == very_long_function(arg1, arg2, ...))
{
if (false == very_long_other_function(even_more_args)
{
do_something();
}
else
{
do_something_else();
}
}
И вот такими многоэтажками забит весь проект.