Opened 8 years ago

Closed 5 years ago

#6512 closed enhancement (wontfix)

CPLString should have a constructor from GByte *

Reported by: Kurt Schwehr Owned by: schwehr
Priority: low Milestone: closed_because_of_github_migration
Component: default Version: svn-trunk
Severity: minor Keywords:
Cc:

Description

If CPLString had a constructor for GByte *, it would allow creation without casting and encourage more use of CPLString rather than bare C strings.

An example from gdaljp2metadatagenerator.cpp:

-    CPLString osTemplate((const char*)pabyStr);
+    CPLString osTemplate(reinterpret_cast<char *>(pabyStr));

Would better be:

-    CPLString osTemplate((const char*)pabyStr);
+    CPLString osTemplate(pabyStr);

Thoughts?

Change History (1)

comment:1 by Even Rouault, 5 years ago

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.