EttusResearch / uhd

The USRP™ Hardware Driver Repository
http://uhd.ettus.com
Other
942 stars 644 forks source link

host/include/uhd/types/bye_vector.hpp: include cstdint #741

Open ffontaine opened 2 months ago

ffontaine commented 2 months ago

Include cstdint to fix the following build failure:

In file included from /home/buildroot/instance-0/output-1/build/uhd-4.3.0.0/host/lib/types/byte_vector.cpp:9: /home/buildroot/instance-0/output-1/build/uhd-4.3.0.0/host/include/uhd/types/byte_vector.hpp:19:21: error: 'uint8_t' was not declared in this scope; did you mean 'boost::uint8_t'?
   19 | typedef std::vector<uint8_t> byte_vector_t;
      |                     ^~~~~~~
      |                     boost::uint8_t

Fixes:

github-actions[bot] commented 2 months ago

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

mbr0wn commented 23 hours ago

Thanks @ffontaine. I started playing around with IWYU which will make this PR obsolete, but this inspired me to give IWYU a try.