So far we have used std::string for asset catalog paths. Some operations
are better described on a dedicated class for this, though.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- temp-asset-catalog-path (branched from master)
- Build Status
Buildable 17464 Build 17464: arc lint + arc unit
Event Timeline
| source/blender/blenkernel/BKE_asset_catalog_path.hh | ||
|---|---|---|
| 47 | Maybe comment on how encoding is handled? Which is I guess the same as std::string (just a byte buffer with no specified encoding), but still good to be explicit about. | |
| source/blender/blenkernel/intern/asset_catalog_path_test.cc | ||
| 67 | Looks weird to compare this to itself. At first I thought this was a mistake, so maybe comment on it? | |
| 92 | Shouldn't this test for empty paths/components? E.g. "" / "something" shouldn't result in "/something", and vice versa. | |
Comment Actions
- Various updates after feedback, including defining SEPARATOR in AssetCatalogPath in a way that can be used by other code. Actually moving the other code to AssetCatalogPath is out of scope for this patch, though.