- 1
- 2
- 3
- 4
- 5
- 6
QByteArray icqMessage::convertToByteArray(const quint8 &d)
{
QByteArray packet;
packet[0] = d;
return packet;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+59
QByteArray icqMessage::convertToByteArray(const quint8 &d)
{
QByteArray packet;
packet[0] = d;
return packet;
}
Обнаружено в сорцах qutim'а. Про memcpy разработчики, видимо, не слышали, также, как и про метод append() в классе QByteArray.
А еще не совсем понятно, зачем функции для конвертирования байт-эррэев в цифры и обратно объявлены и реализованы В КАЖДОМ файле, где используются. Про #include файла, в котором один раз можно реализовать все функции, разработчики, наверное, тоже слышали мельком.
RankoR 23.02.2010 21:51 # 0
Честно говоря, покопавшись в коде, я удивился, как оно вообще работает.
guest 23.02.2010 22:25 # +4.4
guest 24.02.2010 19:42 # +1
правда, указатель придется разыменовывать, но это не беда, ведь любой нормальный компилятор заменит const& на const
RankoR 24.02.2010 20:38 # 0
guest 24.02.2010 00:08 # 0
guest 24.02.2010 01:32 # 0
guest 24.02.2010 18:37 # 0
RankoR 24.02.2010 16:56 # 0
guest 24.02.2010 17:34 # 0
RankoR 24.02.2010 18:24 # 0
guest 24.02.2010 18:39 # +2
Sauron 27.02.2010 01:55 # +1
В общем это причина, по которой мы всё нафиг с нуля переписываем после ухода старого лидера...
RankoR 27.02.2010 16:28 # 0
Sauron 01.03.2010 13:53 # +1