2018-02-25 03:59 GMT

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0002829OpenFOAMPatchpublic2018-02-07 15:58
Reporterhandrake0724 
Assigned Tohenry 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusresolvedResolutionfixed 
Platformx86_64OSArchOS Version(please specify)
Product Version5.x 
Target VersionFixed in Versiondev 
Summary0002829: directFieldMapper has wrong named constructor
DescriptionI found directFieldMapper class has wrong named constructor as below.
I am not sure if this class is in use because the header file is included nowhere.

class directFieldMapper
:
    public FieldMapper
{
    const labelUList& directAddressing_;

    bool hasUnmapped_;

public:

    // Constructors

        //- Construct given addressing
        patchFieldSubset(const labelUList& directAddressing)
        :
            directAddressing_(directAddressing),
            hasUnmapped_(false)
        {
            if (directAddressing_.size() && min(directAddressing_) < 0)
            {
                hasUnmapped_ = true;
            }
        }

    //- Destructor
    virtual ~directFieldMapper()
    {}
    ...
}

Anyway, I think the constructor should be renamed
from
    patchFieldSubset(const labelUList& directAddressing)
to
    directFieldMapper(const labelUList& directAddressing)
TagsNo tags attached.
Attached Files

-Relationships
+Relationships

-Notes

~0009276

henry (manager)

Resolved by commit 8cbde12c80fe8ff7a13aaad8dbc8e7be76e2a894
+Notes

-Issue History
Date Modified Username Field Change
2018-02-07 01:29 handrake0724 New Issue
2018-02-07 15:58 henry Assigned To => henry
2018-02-07 15:58 henry Status new => resolved
2018-02-07 15:58 henry Resolution open => fixed
2018-02-07 15:58 henry Fixed in Version => dev
2018-02-07 15:58 henry Note Added: 0009276
+Issue History