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

Document the RGBA structure #6128

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+14 −12
Open

Document the RGBA structure #6128

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 14 additions & 12 deletions common/include/pcl/impl/point_types.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -342,20 +342,20 @@ namespace pcl
PCL_EXPORTS std::ostream& operator << (std::ostream& os, const RGB& p);
/** \brief A structure representing RGB color information.
*
* The RGBA information is available either as separate r, g, b, or as a
* packed std::uint32_t rgba value. To pack it, use:
* The RGB information is available either as separate r, g, b or as a
* packed std::uint32_t rgb value. To pack it, use:
*
* \code
* int rgb = ((int)r) << 16 | ((int)g) << 8 | ((int)b);
* std::uint32_t rgb = ((int)r) << 16 | ((int)g) << 8 | ((int)b);
* \endcode
*
* To unpack it use:
*
* \code
* int rgb = ...;
* std::uint32_t rgb = p.rgba;
* std::uint8_t r = (rgb >> 16) & 0x0000ff;
* std::uint8_t g = (rgb >> 8) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
* \endcode
*
*/
Expand Down Expand Up @@ -506,20 +506,22 @@ namespace pcl
PCL_EXPORTS std::ostream& operator << (std::ostream& os, const PointXYZRGBA& p);
/** \brief A point structure representing Euclidean xyz coordinates, and the RGBA color.
*
* The RGBA information is available either as separate r, g, b, or as a
* packed std::uint32_t rgba value. To pack it, use:
* The RGBA information is available either as separate r, g, b and a uint8_t values,
* or as a packed std::uint32_t rgba value. To pack it, use:
*
* \code
* int rgb = ((int)r) << 16 | ((int)g) << 8 | ((int)b);
* std::uint32_t rgba = ((std::uint32_t)a << 24 | ((std::uint32_t)r << 16 | ((std::uint32_t)g << 8) | b;
* \endcode
*
* To unpack it use:
*
* \code
* int rgb = ...;
* // unpack rgb into r/g/b
* std::uint32_t rgb = p.rgba;
* std::uint8_t a = (rgb >> 24) & 0x0000ff;
* std::uint8_t r = (rgb >> 16) & 0x0000ff;
* std::uint8_t g = (rgb >> 8) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
* \endcode
*
* \ingroup common
Expand Down Expand Up @@ -579,7 +581,7 @@ namespace pcl
* \code
* PointXYZRGB p;
* // unpack rgb into r/g/b
* std::uint32_t rgb = *reinterpret_cast<int*>(&p.rgb);
* std::uint32_t rgb = p.rgb;
* std::uint8_t r = (rgb >> 16) & 0x0000ff;
* std::uint8_t g = (rgb >> 8) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
Expand Down Expand Up @@ -899,7 +901,7 @@ namespace pcl
* \code
* PointXYZRGB p;
* // unpack rgb into r/g/b
* std::uint32_t rgb = *reinterpret_cast<int*>(&p.rgb);
* std::uint32_t rgb = p.rgba;
* std::uint8_t r = (rgb >> 16) & 0x0000ff;
* std::uint8_t g = (rgb >> 8) & 0x0000ff;
* std::uint8_t b = (rgb) & 0x0000ff;
Expand Down