Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PART IV: Clean up include directives and order under lib/mayaUsd #391

Merged
merged 7 commits into from
Apr 6, 2020
Merged

PART IV: Clean up include directives and order under lib/mayaUsd #391

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
dfc4e31
Fix include directives and order under lib/mayaUsd.
Mar 27, 2020
210a2ab
feedback: Fix header inclusion.
Mar 28, 2020
c05eb7f
put back the missing #ifdef.
Mar 28, 2020
fb592d1
Private project's header files should be #include'd using the file na…
Mar 28, 2020
886119f
UPDATE: include directives and orders under lib/mayaUsd according to …
Apr 1, 2020
db250b3
UPDATE lib/mayaUsd: All included public header files from outside an…
Apr 3, 2020
c4ffc1e
Merge branch 'sabrih/MAYA-104002/clean_up_part_1' into sabrih/MAYA-10…
HamedSabri-adsk Apr 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
2 changes: 1 addition & 1 deletion lib/mayaUsd/base/api.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
// limitations under the License.
//

#include "pxr/base/arch/export.h"
#include <pxr/base/arch/export.h>

#if defined _WIN32 || defined __CYGWIN__

Expand Down
4 changes: 2 additions & 2 deletions lib/mayaUsd/base/debugCodes.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "debugCodes.h"
#include <mayaUsd/base/debugCodes.h>

#include "pxr/base/tf/registryManager.h"
#include <pxr/base/tf/registryManager.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
4 changes: 2 additions & 2 deletions lib/mayaUsd/base/debugCodes.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
#ifndef PXRUSDMAYA_DEBUGCODES_H
#define PXRUSDMAYA_DEBUGCODES_H

#include "pxr/pxr.h"
#include "pxr/base/tf/debug.h"
#include <pxr/pxr.h>
#include <pxr/base/tf/debug.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
2 changes: 1 addition & 1 deletion lib/mayaUsd/fileio/chaser/chaser.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "chaser.h"
#include <mayaUsd/fileio/chaser/chaser.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
12 changes: 5 additions & 7 deletions lib/mayaUsd/fileio/chaser/chaser.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,14 @@
#ifndef PXRUSDMAYA_CHASER_H
#define PXRUSDMAYA_CHASER_H

/// \file usdMaya/chaser.h
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to leave these Doxygen tags if we can. You can truncate it to just this:

/// \file

Same goes for all of the files below.

#include <mayaUsd/base/api.h>

#include "../../base/api.h"
#include <pxr/pxr.h>

#include "pxr/pxr.h"
#include <pxr/base/tf/declarePtrs.h>
#include <pxr/base/tf/refPtr.h>

#include "pxr/base/tf/declarePtrs.h"
#include "pxr/base/tf/refPtr.h"

#include "pxr/usd/usd/timeCode.h"
#include <pxr/usd/usd/timeCode.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
10 changes: 5 additions & 5 deletions lib/mayaUsd/fileio/chaser/chaserRegistry.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,15 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "chaserRegistry.h"
#include <mayaUsd/fileio/chaser/chaserRegistry.h>

#include "../../base/debugCodes.h"
#include "../registryHelper.h"

#include "pxr/base/tf/instantiateSingleton.h"
#include <mayaUsd/base/debugCodes.h>
#include <mayaUsd/fileio/registryHelper.h>

#include <map>

#include <pxr/base/tf/instantiateSingleton.h>

PXR_NAMESPACE_OPEN_SCOPE


Expand Down
20 changes: 9 additions & 11 deletions lib/mayaUsd/fileio/chaser/chaserRegistry.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,20 +16,18 @@
#ifndef PXRUSDMAYA_CHASER_REGISTRY_H
#define PXRUSDMAYA_CHASER_REGISTRY_H

/// \file usdMaya/chaserRegistry.h
#include <mayaUsd/base/api.h>
#include <mayaUsd/fileio/chaser/chaser.h>
#include <mayaUsd/fileio/jobs/jobArgs.h>
#include <mayaUsd/utils/util.h>

#include "../../base/api.h"
#include "chaser.h"
#include "../jobs/jobArgs.h"
#include "../../utils/util.h"
#include <pxr/pxr.h>

#include "pxr/pxr.h"
#include <pxr/base/tf/declarePtrs.h>
#include <pxr/base/tf/registryManager.h>
#include <pxr/base/tf/singleton.h>

#include "pxr/base/tf/declarePtrs.h"
#include "pxr/base/tf/registryManager.h"
#include "pxr/base/tf/singleton.h"

#include "pxr/usd/usd/stage.h"
#include <pxr/usd/usd/stage.h>

#include <functional>

Expand Down
7 changes: 3 additions & 4 deletions lib/mayaUsd/fileio/fallbackPrimReader.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,10 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "fallbackPrimReader.h"
#include <mayaUsd/fileio/fallbackPrimReader.h>
#include <mayaUsd/fileio/translators/translatorUtil.h>

#include "translators/translatorUtil.h"

#include "pxr/usd/usdGeom/imageable.h"
#include <pxr/usd/usdGeom/imageable.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
6 changes: 2 additions & 4 deletions lib/mayaUsd/fileio/fallbackPrimReader.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,8 @@
#ifndef PXRUSDMAYA_FALLBACK_PRIM_READER_H
#define PXRUSDMAYA_FALLBACK_PRIM_READER_H

/// \file usdMaya/fallbackPrimReader.h

#include "primReader.h"
#include "primReaderRegistry.h"
#include <mayaUsd/fileio/primReader.h>
#include <mayaUsd/fileio/primReaderRegistry.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
2 changes: 1 addition & 1 deletion lib/mayaUsd/fileio/functorPrimReader.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "functorPrimReader.h"
#include <mayaUsd/fileio/functorPrimReader.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
16 changes: 7 additions & 9 deletions lib/mayaUsd/fileio/functorPrimReader.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,18 +16,16 @@
#ifndef PXRUSDMAYA_FUNCTORPRIMREADER_H
#define PXRUSDMAYA_FUNCTORPRIMREADER_H

/// \file usdMaya/functorPrimReader.h

#include "../base/api.h"
#include "primReader.h"
#include "primReaderArgs.h"
#include "primReaderContext.h"
#include "primReaderRegistry.h"

#include "pxr/pxr.h"
#include <mayaUsd/base/api.h>
#include <mayaUsd/fileio/primReader.h>
#include <mayaUsd/fileio/primReaderArgs.h>
#include <mayaUsd/fileio/primReaderContext.h>
#include <mayaUsd/fileio/primReaderRegistry.h>

#include <functional>

#include <pxr/pxr.h>

PXR_NAMESPACE_OPEN_SCOPE

/// \class UsdMaya_FunctorPrimReader
Expand Down
27 changes: 12 additions & 15 deletions lib/mayaUsd/fileio/functorPrimWriter.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,28 +13,25 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "pxr/pxr.h"
#include "functorPrimWriter.h"
#include <mayaUsd/fileio/functorPrimWriter.h>
#include <mayaUsd/fileio/primWriter.h>
#include <mayaUsd/fileio/primWriterArgs.h>
#include <mayaUsd/fileio/primWriterContext.h>
#include <mayaUsd/fileio/primWriterRegistry.h>
#include <mayaUsd/fileio/transformWriter.h>
#include <mayaUsd/fileio/writeJobContext.h>

#include "primWriter.h"
#include "primWriterArgs.h"
#include "primWriterContext.h"
#include "primWriterRegistry.h"
#include "transformWriter.h"
#include "writeJobContext.h"

#include "pxr/usd/sdf/path.h"
#include "pxr/usd/usd/stage.h"
#include "pxr/usd/usd/timeCode.h"
#include <functional>

#include <maya/MFnDependencyNode.h>

#include <functional>

#include <pxr/pxr.h>
#include <pxr/usd/sdf/path.h>
#include <pxr/usd/usd/stage.h>
#include <pxr/usd/usd/timeCode.h>

PXR_NAMESPACE_OPEN_SCOPE


UsdMaya_FunctorPrimWriter::UsdMaya_FunctorPrimWriter(
const MFnDependencyNode& depNodeFn,
const SdfPath& usdPath,
Expand Down
21 changes: 8 additions & 13 deletions lib/mayaUsd/fileio/functorPrimWriter.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,26 +16,21 @@
#ifndef PXRUSDMAYA_FUNCTOR_PRIM_WRITER_H
#define PXRUSDMAYA_FUNCTOR_PRIM_WRITER_H

/// \file usdMaya/functorPrimWriter.h
#include <mayaUsd/fileio/transformWriter.h>
#include <mayaUsd/fileio/primWriter.h>
#include <mayaUsd/fileio/primWriterRegistry.h>
#include <mayaUsd/fileio/writeJobContext.h>

#include "pxr/pxr.h"
#include "transformWriter.h"

#include "primWriter.h"
#include "primWriterRegistry.h"
#include "writeJobContext.h"

#include "pxr/usd/sdf/path.h"
#include "pxr/usd/usd/timeCode.h"
#include <functional>

#include <maya/MFnDependencyNode.h>

#include <functional>

#include <pxr/pxr.h>
#include <pxr/usd/sdf/path.h>
#include <pxr/usd/usd/timeCode.h>

PXR_NAMESPACE_OPEN_SCOPE


/// \class UsdMaya_FunctorPrimWriter
/// \brief This class is scaffolding to hold bare prim writer functions and
/// adapt them to the UsdMayaPrimWriter or UsdMayaTransformWriter interface
Expand Down
2 changes: 1 addition & 1 deletion lib/mayaUsd/fileio/importData.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#include "importData.h"
#include <mayaUsd/fileio/importData.h>

#include <type_traits>

Expand Down
8 changes: 4 additions & 4 deletions lib/mayaUsd/fileio/importData.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,15 @@

#pragma once

#include "../base/api.h"
#include <mayaUsd/base/api.h>

#include <map>
#include <string>

#include <pxr/usd/usd/stage.h>
#include <pxr/usd/usd/prim.h>
#include <pxr/usd/sdf/types.h>

#include <map>
#include <string>

PXR_NAMESPACE_USING_DIRECTIVE

MAYAUSD_NS_DEF {
Expand Down
23 changes: 10 additions & 13 deletions lib/mayaUsd/fileio/instancedNodeWriter.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,29 +13,26 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "pxr/pxr.h"
#include "instancedNodeWriter.h"
#include <mayaUsd/fileio/instancedNodeWriter.h>

#include "primWriter.h"
#include "../utils/util.h"
#include "writeJobContext.h"
#include <mayaUsd/fileio/primWriter.h>
#include <mayaUsd/utils/util.h>
#include <mayaUsd/fileio/writeJobContext.h>

#include "pxr/usd/sdf/path.h"
#include "pxr/usd/usd/references.h"
#include "pxr/usd/usd/timeCode.h"
#include <string>
#include <vector>

#include <maya/MDagPath.h>
#include <maya/MDagPathArray.h>

#include <maya/MFnDependencyNode.h>

#include <string>
#include <vector>

#include <pxr/pxr.h>
#include <pxr/usd/sdf/path.h>
#include <pxr/usd/usd/references.h>
#include <pxr/usd/usd/timeCode.h>

PXR_NAMESPACE_OPEN_SCOPE


/// Assuming that \p instance1 and \p instance2 are instances of one another,
/// replaces the prefix \p instance1 in \p dagPath with \p instance2.
static
Expand Down
19 changes: 7 additions & 12 deletions lib/mayaUsd/fileio/instancedNodeWriter.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,25 +16,20 @@
#ifndef PXRUSDMAYA_INSTANCED_NODE_WRITER_H
#define PXRUSDMAYA_INSTANCED_NODE_WRITER_H

/// \file usdMaya/instancedNodeWriter.h
#include <mayaUsd/fileio/primWriter.h>
#include <mayaUsd/utils/util.h>
#include <mayaUsd/fileio/writeJobContext.h>

#include "pxr/pxr.h"
#include "primWriter.h"

#include "../utils/util.h"
#include "writeJobContext.h"

#include "pxr/usd/sdf/path.h"
#include "pxr/usd/usd/timeCode.h"
#include <vector>

#include <maya/MFnDependencyNode.h>

#include <vector>

#include <pxr/pxr.h>
#include <pxr/usd/sdf/path.h>
#include <pxr/usd/usd/timeCode.h>

PXR_NAMESPACE_OPEN_SCOPE


/// This is a "helper" prim writer used internally by UsdMayaWriteJobContext to
/// author nodes that are directly instanced in Maya.
class UsdMaya_InstancedNodeWriter : public UsdMayaPrimWriter
Expand Down
29 changes: 14 additions & 15 deletions lib/mayaUsd/fileio/jobs/jobArgs.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,30 +13,29 @@
// See the License for the specific language governing permissions and
// limitations under the License.
//
#include "jobArgs.h"
#include <mayaUsd/fileio/jobs/jobArgs.h>

#include "../registryHelper.h"
#include "../shading/shadingModeRegistry.h"
#include <mayaUsd/fileio/registryHelper.h>
#include <mayaUsd/fileio/shading/shadingModeRegistry.h>

#include "pxr/base/tf/diagnostic.h"
#include "pxr/base/tf/envSetting.h"
#include "pxr/base/tf/staticTokens.h"
#include "pxr/base/tf/token.h"
#include "pxr/base/vt/dictionary.h"

#include "pxr/usd/sdf/path.h"
#include "pxr/usd/sdf/schema.h"
#include "pxr/usd/usdGeom/tokens.h"
#include "pxr/usd/usdUtils/pipeline.h"
#include <ostream>
#include <string>

#include <maya/MDagPath.h>
#include <maya/MGlobal.h>
#include <maya/MNodeClass.h>
#include <maya/MTypeId.h>

#include <ostream>
#include <string>
#include <pxr/base/tf/diagnostic.h>
#include <pxr/base/tf/envSetting.h>
#include <pxr/base/tf/staticTokens.h>
#include <pxr/base/tf/token.h>
#include <pxr/base/vt/dictionary.h>

#include <pxr/usd/sdf/path.h>
#include <pxr/usd/sdf/schema.h>
#include <pxr/usd/usdGeom/tokens.h>
#include <pxr/usd/usdUtils/pipeline.h>

PXR_NAMESPACE_OPEN_SCOPE

Expand Down
Loading