View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003829 | OpenFOAM | Patch | public | 2022-04-25 10:07 | 2022-04-25 13:20 |
| Reporter | jkau | Assigned To | henry | ||
| Priority | low | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Platform | LInux | OS | Ubuntu | OS Version | 20.04 |
| Product Version | dev | ||||
| Summary | 0003829: the patchInternalField (zero-gradient field) in outletPhaseMeanVelocity could be changed to const reference | ||||
| Description | I think the updateCoeff() function of outletPhaseMeanVelocity patch field, the Uzg vectorField could be changed to a const reference. | ||||
| Tags | No tags attached. | ||||
|
|
Why do you think it can be changed to a const reference? |
|
|
When looking at the code, my understanding is that Uzg is only used for calculating UnzgMean. But if this is wrong, please ignore and close. |
|
|
In the call vectorField Uzg(patchInternalField()); what does patchInternalField() return? Is it a const or non-const reference? |
|
|
hm, looks like a const reference |
|
|
Which definition of patchInternalField() are you looking at and what is the return type? |
|
|
I am looking at virtual tmp<Field<Type>> patchInternalField() const; defined in fvPatchField.H, wich returns tmp<Field<Type>> |
|
|
which is not a const reference. |
|
|
https://cfd.direct/openfoam-training/programming-cfd/ |
|
|
User error |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2022-04-25 10:07 | jkau | New Issue | |
| 2022-04-25 10:19 | henry | Note Added: 0012556 | |
| 2022-04-25 10:38 | jkau | Note Added: 0012557 | |
| 2022-04-25 10:47 | henry | Note Added: 0012558 | |
| 2022-04-25 10:54 | jkau | Note Added: 0012559 | |
| 2022-04-25 10:58 | henry | Note Added: 0012560 | |
| 2022-04-25 11:23 | jkau | Note Added: 0012561 | |
| 2022-04-25 12:33 | henry | Note Added: 0012562 | |
| 2022-04-25 13:20 | henry | Note Added: 0012563 | |
| 2022-04-25 13:20 | henry | Assigned To | => henry |
| 2022-04-25 13:20 | henry | Status | new => closed |
| 2022-04-25 13:20 | henry | Resolution | open => no change required |
| 2022-04-25 13:20 | henry | Note Added: 0012564 |
