suspicious code in writeFields.C

around Line 615:

    forAll(fc, i)
    {
        const label meshFacei = fvp.patch().start();
        cellFld[fc[i]] = min(cellFld[fc[i]], ownPyrVol[meshFacei]);
    }

Looks like it should be meshFacei = fvp.patch().start() + i` was forgotten. Or instead:

    label meshFacei = fvp.patch().start();

    for (const label celli : fc)
    {
        cellFld[celli] = Foam::min(cellFld[celli], ownPyrVol[meshFacei]);
        ++meshFacei;
    }

seen while looking at #3348

assigned to @Mattijs

changed the description

@Mattijs Other suspicious (bad) code. Finally more closely examined the compiler warning about CoNum being not initialized. The compiler is right.

Can discuss off-line.

mentioned in commit 3b397100

mentioned in commit 6ef21ccc

mentioned in commit 5cf80f96

mentioned in commit 0be19b7f