From 6410d1d37c1ea9d1d168840a7ba063facb0bc9d6 Mon Sep 17 00:00:00 2001 From: Kieran Bingham Date: Fri, 25 Jun 2021 00:09:15 +0100 Subject: libcamera/base: Move class helpers to the base library Move the class support infrastructure to the base library. Reviewed-by: Paul Elder Reviewed-by: Hirokazu Honda Reviewed-by: Laurent Pinchart Signed-off-by: Kieran Bingham --- include/libcamera/base/class.h | 105 +++++++++++++ include/libcamera/base/meson.build | 1 + include/libcamera/buffer.h | 3 +- include/libcamera/camera.h | 3 +- include/libcamera/camera_manager.h | 3 +- include/libcamera/class.h | 105 ------------- include/libcamera/controls.h | 3 +- include/libcamera/framebuffer_allocator.h | 2 +- include/libcamera/internal/buffer.h | 3 +- include/libcamera/internal/byte_stream_buffer.h | 3 +- include/libcamera/internal/camera_sensor.h | 3 +- include/libcamera/internal/file.h | 3 +- include/libcamera/internal/log.h | 3 +- include/libcamera/internal/media_object.h | 2 +- include/libcamera/internal/pipeline_handler.h | 3 +- include/libcamera/internal/v4l2_subdevice.h | 3 +- include/libcamera/internal/v4l2_videodevice.h | 3 +- include/libcamera/meson.build | 1 - include/libcamera/request.h | 3 +- src/android/camera_buffer.h | 3 +- src/android/camera_capabilities.h | 3 +- src/android/camera_hal_config.h | 2 +- src/libcamera/base/class.cpp | 190 ++++++++++++++++++++++++ src/libcamera/base/meson.build | 1 + src/libcamera/class.cpp | 190 ------------------------ src/libcamera/meson.build | 1 - 26 files changed, 329 insertions(+), 316 deletions(-) create mode 100644 include/libcamera/base/class.h delete mode 100644 include/libcamera/class.h create mode 100644 src/libcamera/base/class.cpp delete mode 100644 src/libcamera/class.cpp diff --git a/include/libcamera/base/class.h b/include/libcamera/base/class.h new file mode 100644 index 00000000..a07dac05 --- /dev/null +++ b/include/libcamera/base/class.h @@ -0,0 +1,105 @@ +/* SPDX-License-Identifier: LGPL-2.1-or-later */ +/* + * Copyright (C) 2020, Google Inc. + * + * class.h - Utilities and helpers for classes + */ +#ifndef __LIBCAMERA_BASE_CLASS_H__ +#define __LIBCAMERA_BASE_CLASS_H__ + +#include + +namespace libcamera { + +#ifndef __DOXYGEN__ +#define LIBCAMERA_DISABLE_COPY(klass) \ + klass(const klass &) = delete; \ + klass &operator=(const klass &) = delete; + +#define LIBCAMERA_DISABLE_MOVE(klass) \ + klass(klass &&) = delete; \ + klass &operator=(klass &&) = delete; + +#define LIBCAMERA_DISABLE_COPY_AND_MOVE(klass) \ + LIBCAMERA_DISABLE_COPY(klass) \ + LIBCAMERA_DISABLE_MOVE(klass) +#else +#define LIBCAMERA_DISABLE_COPY(klass) +#define LIBCAMERA_DISABLE_MOVE(klass) +#define LIBCAMERA_DISABLE_COPY_AND_MOVE(klass) +#endif + +#ifndef __DOXYGEN__ +#define LIBCAMERA_DECLARE_PRIVATE() \ +public: \ + class Private; \ + friend class Private; + +#define LIBCAMERA_DECLARE_PUBLIC(klass) \ + friend class klass; \ + using Public = klass; + +#define LIBCAMERA_D_PTR() \ + _d(); + +#define LIBCAMERA_O_PTR() \ + _o(); + +#else +#define LIBCAMERA_DECLARE_PRIVATE() +#define LIBCAMERA_DECLARE_PUBLIC(klass) +#define LIBCAMERA_D_PTR() +#define LIBCAMERA_O_PTR() +#endif + +class Extensible +{ +public: + class Private + { + public: + Private(Extensible *o); + virtual ~Private(); + +#ifndef __DOXYGEN__ + template + const T *_o() const + { + return static_cast(o_); + } + + template + T *_o() + { + return static_cast(o_); + } +#endif + + private: + Extensible *const o_; + }; + + Extensible(Private *d); + +protected: +#ifndef __DOXYGEN__ + template + const T *_d() const + { + return static_cast(d_.get()); + } + + template + T *_d() + { + return static_cast(d_.get()); + } +#endif + +private: + const std::unique_ptr d_; +}; + +} /* namespace libcamera */ + +#endif /* __LIBCAMERA_BASE_CLASS_H__ */ diff --git a/include/libcamera/base/meson.build b/include/libcamera/base/meson.build index 9f0ba6b0..2db756c5 100644 --- a/include/libcamera/base/meson.build +++ b/include/libcamera/base/meson.build @@ -3,6 +3,7 @@ libcamera_base_include_dir = libcamera_include_dir / 'base' libcamera_base_headers = files([ + 'class.h', 'utils.h', ]) diff --git a/include/libcamera/buffer.h b/include/libcamera/buffer.h index e0af0090..323d1cba 100644 --- a/include/libcamera/buffer.h +++ b/include/libcamera/buffer.h @@ -10,7 +10,8 @@ #include #include -#include +#include + #include namespace libcamera { diff --git a/include/libcamera/camera.h b/include/libcamera/camera.h index d7164180..ea091400 100644 --- a/include/libcamera/camera.h +++ b/include/libcamera/camera.h @@ -12,7 +12,8 @@ #include #include -#include +#include + #include #include #include diff --git a/include/libcamera/camera_manager.h b/include/libcamera/camera_manager.h index c2f0b786..5deede03 100644 --- a/include/libcamera/camera_manager.h +++ b/include/libcamera/camera_manager.h @@ -12,7 +12,8 @@ #include #include -#include +#include + #include #include diff --git a/include/libcamera/class.h b/include/libcamera/class.h deleted file mode 100644 index f384a488..00000000 --- a/include/libcamera/class.h +++ /dev/null @@ -1,105 +0,0 @@ -/* SPDX-License-Identifier: LGPL-2.1-or-later */ -/* - * Copyright (C) 2020, Google Inc. - * - * class.h - Utilities and helpers for classes - */ -#ifndef __LIBCAMERA_CLASS_H__ -#define __LIBCAMERA_CLASS_H__ - -#include - -namespace libcamera { - -#ifndef __DOXYGEN__ -#define LIBCAMERA_DISABLE_COPY(klass) \ - klass(const klass &) = delete; \ - klass &operator=(const klass &) = delete; - -#define LIBCAMERA_DISABLE_MOVE(klass) \ - klass(klass &&) = delete; \ - klass &operator=(klass &&) = delete; - -#define LIBCAMERA_DISABLE_COPY_AND_MOVE(klass) \ - LIBCAMERA_DISABLE_COPY(klass) \ - LIBCAMERA_DISABLE_MOVE(klass) -#else -#define LIBCAMERA_DISABLE_COPY(klass) -#define LIBCAMERA_DISABLE_MOVE(klass) -#define LIBCAMERA_DISABLE_COPY_AND_MOVE(klass) -#endif - -#ifndef __DOXYGEN__ -#define LIBCAMERA_DECLARE_PRIVATE() \ -public: \ - class Private; \ - friend class Private; - -#define LIBCAMERA_DECLARE_PUBLIC(klass) \ - friend class klass; \ - using Public = klass; - -#define LIBCAMERA_D_PTR() \ - _d(); - -#define LIBCAMERA_O_PTR() \ - _o(); - -#else -#define LIBCAMERA_DECLARE_PRIVATE() -#define LIBCAMERA_DECLARE_PUBLIC(klass) -#define LIBCAMERA_D_PTR() -#define LIBCAMERA_O_PTR() -#endif - -class Extensible -{ -public: - class Private - { - public: - Private(Extensible *o); - virtual ~Private(); - -#ifndef __DOXYGEN__ - template - const T *_o() const - { - return static_cast(o_); - } - - template - T *_o() - { - return static_cast(o_); - } -#endif - - private: - Extensible *const o_; - }; - - Extensible(Private *d); - -protected: -#ifndef __DOXYGEN__ - template - const T *_d() const - { - return static_cast(d_.get()); - } - - template - T *_d() - { - return static_cast(d_.get()); - } -#endif - -private: - const std::unique_ptr d_; -}; - -} /* namespace libcamera */ - -#endif /* __LIBCAMERA_CLASS_H__ */ diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h index 1c9b37e6..f62b6cf0 100644 --- a/include/libcamera/controls.h +++ b/include/libcamera/controls.h @@ -14,7 +14,8 @@ #include #include -#include +#include + #include #include diff --git a/include/libcamera/framebuffer_allocator.h b/include/libcamera/framebuffer_allocator.h index 0c85631a..cbc9ce10 100644 --- a/include/libcamera/framebuffer_allocator.h +++ b/include/libcamera/framebuffer_allocator.h @@ -11,7 +11,7 @@ #include #include -#include +#include namespace libcamera { diff --git a/include/libcamera/internal/buffer.h b/include/libcamera/internal/buffer.h index 9da1fbd1..91dd2496 100644 --- a/include/libcamera/internal/buffer.h +++ b/include/libcamera/internal/buffer.h @@ -10,7 +10,8 @@ #include #include -#include +#include + #include #include diff --git a/include/libcamera/internal/byte_stream_buffer.h b/include/libcamera/internal/byte_stream_buffer.h index 866cb9b0..7eefb1a7 100644 --- a/include/libcamera/internal/byte_stream_buffer.h +++ b/include/libcamera/internal/byte_stream_buffer.h @@ -11,7 +11,8 @@ #include #include -#include +#include + #include namespace libcamera { diff --git a/include/libcamera/internal/camera_sensor.h b/include/libcamera/internal/camera_sensor.h index e133ebf4..7bc54061 100644 --- a/include/libcamera/internal/camera_sensor.h +++ b/include/libcamera/internal/camera_sensor.h @@ -11,7 +11,8 @@ #include #include -#include +#include + #include #include #include diff --git a/include/libcamera/internal/file.h b/include/libcamera/internal/file.h index f0b313a5..44621ceb 100644 --- a/include/libcamera/internal/file.h +++ b/include/libcamera/internal/file.h @@ -11,7 +11,8 @@ #include #include -#include +#include + #include namespace libcamera { diff --git a/include/libcamera/internal/log.h b/include/libcamera/internal/log.h index 9c2beab6..82e55a62 100644 --- a/include/libcamera/internal/log.h +++ b/include/libcamera/internal/log.h @@ -10,8 +10,7 @@ #include #include -#include - +#include #include namespace libcamera { diff --git a/include/libcamera/internal/media_object.h b/include/libcamera/internal/media_object.h index 1c82c279..2f5d33e1 100644 --- a/include/libcamera/internal/media_object.h +++ b/include/libcamera/internal/media_object.h @@ -12,7 +12,7 @@ #include -#include +#include namespace libcamera { diff --git a/include/libcamera/internal/pipeline_handler.h b/include/libcamera/internal/pipeline_handler.h index 31dadf28..8beb6b76 100644 --- a/include/libcamera/internal/pipeline_handler.h +++ b/include/libcamera/internal/pipeline_handler.h @@ -15,7 +15,8 @@ #include #include -#include +#include + #include #include #include diff --git a/include/libcamera/internal/v4l2_subdevice.h b/include/libcamera/internal/v4l2_subdevice.h index d2b9ca55..d07dd6b4 100644 --- a/include/libcamera/internal/v4l2_subdevice.h +++ b/include/libcamera/internal/v4l2_subdevice.h @@ -11,7 +11,8 @@ #include #include -#include +#include + #include #include "libcamera/internal/formats.h" diff --git a/include/libcamera/internal/v4l2_videodevice.h b/include/libcamera/internal/v4l2_videodevice.h index 7938343b..227d015e 100644 --- a/include/libcamera/internal/v4l2_videodevice.h +++ b/include/libcamera/internal/v4l2_videodevice.h @@ -16,8 +16,9 @@ #include +#include + #include -#include #include #include #include diff --git a/include/libcamera/meson.build b/include/libcamera/meson.build index 4ac864b7..21a43388 100644 --- a/include/libcamera/meson.build +++ b/include/libcamera/meson.build @@ -5,7 +5,6 @@ libcamera_public_headers = files([ 'buffer.h', 'camera.h', 'camera_manager.h', - 'class.h', 'compiler.h', 'controls.h', 'file_descriptor.h', diff --git a/include/libcamera/request.h b/include/libcamera/request.h index 5596901d..00c646fe 100644 --- a/include/libcamera/request.h +++ b/include/libcamera/request.h @@ -13,7 +13,8 @@ #include #include -#include +#include + #include #include diff --git a/src/android/camera_buffer.h b/src/android/camera_buffer.h index c88124b2..e850c4e3 100644 --- a/src/android/camera_buffer.h +++ b/src/android/camera_buffer.h @@ -9,7 +9,8 @@ #include -#include +#include + #include class CameraBuffer final : public libcamera::Extensible diff --git a/src/android/camera_capabilities.h b/src/android/camera_capabilities.h index f511607b..4f5be825 100644 --- a/src/android/camera_capabilities.h +++ b/src/android/camera_capabilities.h @@ -11,8 +11,9 @@ #include #include +#include + #include -#include #include #include diff --git a/src/android/camera_hal_config.h b/src/android/camera_hal_config.h index 97dc69c1..a79d5d6c 100644 --- a/src/android/camera_hal_config.h +++ b/src/android/camera_hal_config.h @@ -10,7 +10,7 @@ #include #include -#include +#include struct CameraConfigData { int facing = -1; diff --git a/src/libcamera/base/class.cpp b/src/libcamera/base/class.cpp new file mode 100644 index 00000000..165beafc --- /dev/null +++ b/src/libcamera/base/class.cpp @@ -0,0 +1,190 @@ +/* SPDX-License-Identifier: LGPL-2.1-or-later */ +/* + * Copyright (C) 2020, Google Inc. + * + * class.cpp - Utilities and helpers for classes + */ + +#include + +/** + * \file class.h + * \brief Utilities to help constructing class interfaces + * + * The extensible class can be inherited to create public classes with stable + * ABIs. + */ + +namespace libcamera { + +/** + * \def LIBCAMERA_DISABLE_COPY + * \brief Disable copy construction and assignment of the \a klass + * \param klass The name of the class + * + * Example usage: + * \code{.cpp} + * class NonCopyable + * { + * public: + * NonCopyable(); + * ... + * + * private: + * LIBCAMERA_DISABLE_COPY(NonCopyable) + * }; + * \endcode + */ + +/** + * \def LIBCAMERA_DISABLE_MOVE + * \brief Disable move construction and assignment of the \a klass + * \param klass The name of the class + * + * Example usage: + * \code{.cpp} + * class NonMoveable + * { + * public: + * NonMoveable(); + * ... + * + * private: + * LIBCAMERA_DISABLE_MOVE(NonMoveable) + * }; + * \endcode + */ + +/** + * \def LIBCAMERA_DISABLE_COPY_AND_MOVE + * \brief Disable copy and move construction and assignment of the \a klass + * \param klass The name of the class + * + * Example usage: + * \code{.cpp} + * class NonCopyableNonMoveable + * { + * public: + * NonCopyableNonMoveable(); + * ... + * + * private: + * LIBCAMERA_DISABLE_COPY_AND_MOVE(NonCopyableNonMoveable) + * }; + * \endcode + */ + +/** + * \def LIBCAMERA_DECLARE_PRIVATE + * \brief Declare private data for a public class + * + * The LIBCAMERA_DECLARE_PRIVATE() macro plumbs the infrastructure necessary to + * make a class manage its private data through a d-pointer. It shall be used at + * the very top of the class definition. + */ + +/** + * \def LIBCAMERA_DECLARE_PUBLIC + * \brief Declare public data for a private class + * \param klass The public class name + * + * The LIBCAMERA_DECLARE_PUBLIC() macro is the counterpart of + * LIBCAMERA_DECLARE_PRIVATE() to be used in the private data class. It shall be + * used at the very top of the private class definition, with the public class + * name passed as the \a klass parameter. + */ + +/** + * \def LIBCAMERA_D_PTR() + * \brief Retrieve the private data pointer + * + * This macro can be used in any member function of a class that inherits, + * directly or indirectly, from the Extensible class, to create a local + * variable named 'd' that points to the class' private data instance. + */ + +/** + * \def LIBCAMERA_O_PTR() + * \brief Retrieve the public instance corresponding to the private data + * + * This macro is the counterpart of LIBCAMERA_D_PTR() for private data classes. + * It can be used in any member function of the private data class to create a + * local variable named 'o' that points to the public class instance + * corresponding to the private data. + */ + +/** + * \class Extensible + * \brief Base class to manage private data through a d-pointer + * + * The Extensible class provides a base class to implement the + * d-pointer design pattern (also + * known as opaque pointer + * or pImpl idiom). + * It helps creating public classes that can be extended without breaking their + * ABI. Such classes store their private data in a separate private data object, + * referenced by a pointer in the public class (hence the name d-pointer). + * + * Classes that follow this design pattern are referred herein as extensible + * classes. To be extensible, a class PublicClass shall: + * + * - inherit from the Extensible class or from another extensible class + * - invoke the LIBCAMERA_DECLARE_PRIVATE() macro at the very top of the class + * definition + * - define a private data class named PublicClass::Private that inherits from + * the Private data class of the base class + * - invoke the LIBCAMERA_DECLARE_PUBLIC() macro at the very top of the Private + * data class definition + * - pass a pointer to a newly allocated Private data object to the constructor + * of the base class + * + * Additionally, if the PublicClass is not final, it shall expose one or more + * constructors that takes a pointer to a Private data instance, to be used by + * derived classes. + * + * The Private class is fully opaque to users of the libcamera public API. + * Internally, it can be kept private to the implementation of PublicClass, or + * be exposed to other classes. In the latter case, the members of the Private + * class need to be qualified with appropriate access specifiers. The + * PublicClass and Private classes always have full access to each other's + * protected and private members. + */ + +/** + * \brief Construct an instance of an Extensible class + * \param[in] d Pointer to the private data instance + */ +Extensible::Extensible(Extensible::Private *d) + : d_(d) +{ +} + +/** + * \var Extensible::d_ + * \brief Pointer to the private data instance + */ + +/** + * \class Extensible::Private + * \brief Base class for private data managed through a d-pointer + */ + +/** + * \brief Construct an instance of an Extensible class private data + * \param[in] o Pointer to the public class object + */ +Extensible::Private::Private(Extensible *o) + : o_(o) +{ +} + +Extensible::Private::~Private() +{ +} + +/** + * \var Extensible::Private::o_ + * \brief Pointer to the public class object + */ + +} /* namespace libcamera */ diff --git a/src/libcamera/base/meson.build b/src/libcamera/base/meson.build index 6d9ec374..302a2886 100644 --- a/src/libcamera/base/meson.build +++ b/src/libcamera/base/meson.build @@ -1,6 +1,7 @@ # SPDX-License-Identifier: CC0-1.0 libcamera_base_sources = files([ + 'class.cpp', 'utils.cpp', ]) diff --git a/src/libcamera/class.cpp b/src/libcamera/class.cpp deleted file mode 100644 index 28c35633..00000000 --- a/src/libcamera/class.cpp +++ /dev/null @@ -1,190 +0,0 @@ -/* SPDX-License-Identifier: LGPL-2.1-or-later */ -/* - * Copyright (C) 2020, Google Inc. - * - * class.cpp - Utilities and helpers for classes - */ - -#include - -/** - * \file class.h - * \brief Utilities to help constructing class interfaces - * - * The extensible class can be inherited to create public classes with stable - * ABIs. - */ - -namespace libcamera { - -/** - * \def LIBCAMERA_DISABLE_COPY - * \brief Disable copy construction and assignment of the \a klass - * \param klass The name of the class - * - * Example usage: - * \code{.cpp} - * class NonCopyable - * { - * public: - * NonCopyable(); - * ... - * - * private: - * LIBCAMERA_DISABLE_COPY(NonCopyable) - * }; - * \endcode - */ - -/** - * \def LIBCAMERA_DISABLE_MOVE - * \brief Disable move construction and assignment of the \a klass - * \param klass The name of the class - * - * Example usage: - * \code{.cpp} - * class NonMoveable - * { - * public: - * NonMoveable(); - * ... - * - * private: - * LIBCAMERA_DISABLE_MOVE(NonMoveable) - * }; - * \endcode - */ - -/** - * \def LIBCAMERA_DISABLE_COPY_AND_MOVE - * \brief Disable copy and move construction and assignment of the \a klass - * \param klass The name of the class - * - * Example usage: - * \code{.cpp} - * class NonCopyableNonMoveable - * { - * public: - * NonCopyableNonMoveable(); - * ... - * - * private: - * LIBCAMERA_DISABLE_COPY_AND_MOVE(NonCopyableNonMoveable) - * }; - * \endcode - */ - -/** - * \def LIBCAMERA_DECLARE_PRIVATE - * \brief Declare private data for a public class - * - * The LIBCAMERA_DECLARE_PRIVATE() macro plumbs the infrastructure necessary to - * make a class manage its private data through a d-pointer. It shall be used at - * the very top of the class definition. - */ - -/** - * \def LIBCAMERA_DECLARE_PUBLIC - * \brief Declare public data for a private class - * \param klass The public class name - * - * The LIBCAMERA_DECLARE_PUBLIC() macro is the counterpart of - * LIBCAMERA_DECLARE_PRIVATE() to be used in the private data class. It shall be - * used at the very top of the private class definition, with the public class - * name passed as the \a klass parameter. - */ - -/** - * \def LIBCAMERA_D_PTR() - * \brief Retrieve the private data pointer - * - * This macro can be used in any member function of a class that inherits, - * directly or indirectly, from the Extensible class, to create a local - * variable named 'd' that points to the class' private data instance. - */ - -/** - * \def LIBCAMERA_O_PTR() - * \brief Retrieve the public instance corresponding to the private data - * - * This macro is the counterpart of LIBCAMERA_D_PTR() for private data classes. - * It can be used in any member function of the private data class to create a - * local variable named 'o' that points to the public class instance - * corresponding to the private data. - */ - -/** - * \class Extensible - * \brief Base class to manage private data through a d-pointer - * - * The Extensible class provides a base class to implement the - * d-pointer design pattern (also - * known as opaque pointer - * or pImpl idiom). - * It helps creating public classes that can be extended without breaking their - * ABI. Such classes store their private data in a separate private data object, - * referenced by a pointer in the public class (hence the name d-pointer). - * - * Classes that follow this design pattern are referred herein as extensible - * classes. To be extensible, a class PublicClass shall: - * - * - inherit from the Extensible class or from another extensible class - * - invoke the LIBCAMERA_DECLARE_PRIVATE() macro at the very top of the class - * definition - * - define a private data class named PublicClass::Private that inherits from - * the Private data class of the base class - * - invoke the LIBCAMERA_DECLARE_PUBLIC() macro at the very top of the Private - * data class definition - * - pass a pointer to a newly allocated Private data object to the constructor - * of the base class - * - * Additionally, if the PublicClass is not final, it shall expose one or more - * constructors that takes a pointer to a Private data instance, to be used by - * derived classes. - * - * The Private class is fully opaque to users of the libcamera public API. - * Internally, it can be kept private to the implementation of PublicClass, or - * be exposed to other classes. In the latter case, the members of the Private - * class need to be qualified with appropriate access specifiers. The - * PublicClass and Private classes always have full access to each other's - * protected and private members. - */ - -/** - * \brief Construct an instance of an Extensible class - * \param[in] d Pointer to the private data instance - */ -Extensible::Extensible(Extensible::Private *d) - : d_(d) -{ -} - -/** - * \var Extensible::d_ - * \brief Pointer to the private data instance - */ - -/** - * \class Extensible::Private - * \brief Base class for private data managed through a d-pointer - */ - -/** - * \brief Construct an instance of an Extensible class private data - * \param[in] o Pointer to the public class object - */ -Extensible::Private::Private(Extensible *o) - : o_(o) -{ -} - -Extensible::Private::~Private() -{ -} - -/** - * \var Extensible::Private::o_ - * \brief Pointer to the public class object - */ - -} /* namespace libcamera */ diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build index b70688d8..e1252ce4 100644 --- a/src/libcamera/meson.build +++ b/src/libcamera/meson.build @@ -10,7 +10,6 @@ libcamera_sources = files([ 'camera_manager.cpp', 'camera_sensor.cpp', 'camera_sensor_properties.cpp', - 'class.cpp', 'controls.cpp', 'control_serializer.cpp', 'control_validator.cpp', -- cgit v1.2.1