Can we please merge this simple fix?

https://github.com/sigrokproject/pulseview/pull/50

On Wed, Sep 14, 2022 at 10:13 AM Paul Kasemir <paul.kase...@gmail.com>
wrote:

> Hi,
>
> Can we merge this simple patch from
> https://github.com/sigrokproject/pulseview/pull/50. Without it my windows
> PulseView cross build is failing.
>
> ========================================
> From e808b2ffdac8e0ec8bf02ec807190a158555a301 Mon Sep 17 00:00:00 2001
> From: Paul Kasemir <paul.kase...@gmail.com>
> Date: Tue, 30 Aug 2022 12:32:49 -0600
> Subject: [PATCH] Include <memory> so we don't get error at compile time.
>
> Example build failure:
>   In file included from
> /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.cpp:21:
>
> /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.hpp:130:14:
> error: 'shared_ptr' in namespace 'std' does not name a template type
>     130 | typedef std::shared_ptr<pv::data::Segment> SharedPtrToSegment;
>         |              ^~~~~~~~~~
>
> /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.hpp:32:1:
> note: 'std::shared_ptr' is defined in header '<memory>'; did you forget to
> '#include <memory>'?
>      31 | #include <QObject>
>     +++ |+#include <memory>
>      32 |
> ---
>  pv/data/segment.hpp | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/pv/data/segment.hpp b/pv/data/segment.hpp
> index 66085faf..f4601d04 100644
> --- a/pv/data/segment.hpp
> +++ b/pv/data/segment.hpp
> @@ -24,6 +24,7 @@
>  #include "pv/util.hpp"
>
>  #include <atomic>
> +#include <memory>
>  #include <mutex>
>  #include <thread>
>  #include <deque>
> --
> 2.37.3
> ========================================
>
> --
> Engineering Power - A glowing pool of excellence.
> Lets take a swim.
>


-- 
Engineering Power - A glowing pool of excellence.
Lets take a swim.
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to