Går detta att göra effektivare? (C/C++)

Tjena!

Sitter här och skriver lite kod och har en bit kod som funkar men som kanske inte är så jätteeffektiv. Tycker det ser ganska fult ut, men jag kan inte komma på hur sjutton man skulle kunna fixa det. Tänkte höra om det är nån som är sugen på att klura lite tillsammans med mig.

Om ni inte har så bra koll på color spaces och sånt så försöker jag förklara så gott jag kan vad det är jag försöker göra

Programmet sparar JPG-filer från en inkommande videoström med libjpeg. Den inkommande datan är kodad i UYVY, och jag skriver den med "jpeg_write_raw_data" för att slippa en del onödiga beräkningar. UYVY är redan downsamplat till 4:2:2 så det funkar ju rätt bra att stoppa rätt in i jpeg-codecen.

Problemet är att libjpeg inte gärna tar den råa datan som den är (UYVYUYVYUYVY...osv), utan vill ha den i fina små block med 16 rader i taget. Datan ska där ligga i fina arrays, först en med alla Y, sen en med alla V, och sist med alla U. Det enda sättet jag lyckas mata libjpeg på detta sätt är med ett gäng mindre roliga for-loopar. Här kommer iaf den relevanta koden:

    jpeg_start_compress(&cinfo, true);

    JSAMPROW yv[16],cb[16],cr[16];
    JSAMPARRAY data[3];

    data[0]=yv;
    data[1]=cb;
    data[2]=cr;

    for (int i=0; i<height; i+=16){
        for (int y=0; y<16; y++){
            JSAMPLE* ysamp = new JSAMPLE[width];
            JSAMPLE* cbsamp = new JSAMPLE[width/2];
            JSAMPLE* crsamp = new JSAMPLE[width/2];
            yv[y]=ysamp;
            cb[y]=cbsamp;
            cr[y]=crsamp;
            for (int x = 0; x<width; x++){
                ysamp[x]=*(imageData+(2*x)+1+((width)*2*(y+i))*(1+interlace));
                if(x<width/2){
                    cbsamp[x]=*(imageData+(4*x)+((width)*2*(y+i))*(1+interlace));
                    crsamp[x]=*(imageData+(4*x)+2+((width)*2*(y+i))*(1+interlace));
                }
            }

        }
        jpeg_write_raw_data(&cinfo, data, 16);
        for (int y=0; y<16; y++){
            delete yv[y];
            delete cb[y];
            delete cr[y];
        }
    }

    jpeg_finish_compress(&cinfo);

imageData är en pointer till den råa UYVY-datan som jag använder lite aritmetik på för att stoppa rätt värde i arrayerna.

Är det for-loopar som gäller i såna här tillfällen? Känns ju så segt att behöva gå igenom varje pixel såhär och klippa och klistra som tusan. Finns det inte något bra sätt att skapa arrayer snabbt genom att ta exempelvis varannat värde från en char* ?

Tacksam för all input jag kan få. Har grubblat halva dagen

Ska ta och kika på det. Är lite newbie på sån lågnivåprogrammering, på universitetet var det java för hela slanten med fokus på objektdesignen. Borde läst mer riktig programmering Jaja, det får man ta igen på egen hand såhär när man är klar med pluggandet. Kul att lära sig.

Har flyttat ut alla new och delete nu iaf så de görs en gång istället. Det blev lite bättre Borde jag sett själv XD