吐个槽，工作很久的同事，代码质量仍然像小学生。

myworkstation

回复 10# VIP_fuck


    满满的正能量啊，就当如此。力所能及的把事件做好。

VIP_fuck

回复 11# myworkstation


    拿了公司的钱，咱也不能就这么混日子。

    也想做个合格的工程人员。

    the fucking code

zylthinking

VIP_fuck 发表于 2015-08-14 10:30
比较无语，匪夷所思的命名，混乱的编码，一天竟然都不懂他一个函数的实际工作，，，

编码规范难道不应该 ...

幼儿园的路过。。。。。。。， 算了， 不标颜色了， 太多了， 这是我写的， 我写的， 我写的。。。。。。。。。。 那个紫色的， 是google的， 是google的， 是google的。。。。。
就是为了让英语党看不懂， 你信不信？

static uintptr_t chunk_size(video_format* fmt, sp<MetaData> meta, MediaBuffer* buffer,
                            intptr_t chunk0[3][2], intptr_t chunk1[3][2])
{
    int32_t w0 = fmt->pixel->size->width;
    int32_t h0 = fmt->pixel->size->height;
    int32_t nb0 = 0;

    int32_t w1, h1, nb1 = 0;
    meta->findInt32(kKeyWidth, &w1);
    meta->findInt32(kKeyHeight, &h1);

    intptr_t chunk[3][2], (*chunk2)[2] = NULL;
    int32_t w2, h2, nb2 = 0;
    if (0 != ((w1 | h1) & 15)) {
        w2 = roundup(w1, 16);
        h2 = roundup(h1, 16);
        chunk2 = &chunk[0];
    }

    for (int i = 0; i < fmt->pixel->panels; ++i) {
        chunk1[0] = panel_width(i, w1, fmt->pixel->csp);
        chunk1[0] = pixels_to_bytes(i, chunk1[0], fmt->pixel->csp);
        chunk1[1] = panel_height(i, h1, fmt->pixel->csp);
        nb1 += chunk1[0] * chunk1[1];

        if (chunk2 != NULL) {
            chunk2[0] = panel_width(i, w2, fmt->pixel->csp);
            chunk2[0] = pixels_to_bytes(i, chunk2[0], fmt->pixel->csp);
            chunk2[1] = panel_height(i, h2, fmt->pixel->csp);
            nb2 += chunk2[0] * chunk2[1];
        }

        chunk0[0] = panel_width(i, w0, fmt->pixel->csp);
        chunk0[0] = pixels_to_bytes(i, chunk0[0], fmt->pixel->csp);
        chunk0[1] = panel_height(i, h0, fmt->pixel->csp);

        nb0 += chunk0[0] * chunk0[1];
    }

    int32_t nb = (int32_t) buffer->range_length();
    if (nb == nb2) {
        chunk1 = chunk;
    } else if (nb != nb1) {
        logmsg_d("nb mismatch %d:%d:%d\n", nb, nb1, nb2);
    }

    if (chunk1[0] != chunk0[0] || chunk1[1] != chunk0[1]) {
        logmsg_d("pic mismatch %d:%d vs %d:%d\n", chunk1[0], chunk1[1], chunk0[0], chunk0[1]);
    }
    return nb0;
}

csumck

楼上代码挺好的啊  再来点注释就完美了  为啥说这是幼儿园

lz小骆驼

其实人家就是为了让你看不懂  

lz小骆驼

以后只有自己能维护

VIP_fuck

我去。。。

sonicling

记得有本书说代码要自解释，就是说代码写的不需要注释就一清二楚。以前还比较认同，现在只觉得：

这种浪费生命的简单逻辑你找幼儿园小朋友帮你码就行了，还用我出马吗？

ctrigger

lz小骆驼 发表于 2015-08-14 12:21
其实人家就是为了让你看不懂

哈哈，被你看破了

cokeboL

好屌