API change - support more const arg

As dm_report_field_string() doesn't modify content of data pointer,
it can be marked as const.

It's slight API change - but doesn't require any change on the user side
and supports wider range of arguments without const casting.
(i.e. we may use as paramater const lv struct this way:  &lv->name)

diff --git a/WHATS_NEW_DM b/WHATS_NEW_DM
index 004b225d18c61fefd7d25b9cd45c3fadcb6c062a..234989847354c1b3709f2d1e0e2ca25750006d4b 100644 (file)
--- a/WHATS_NEW_DM
+++ b/WHATS_NEW_DM
@@ -1,5 +1,6 @@
 Version 1.02.64 - 
 ===================================
+  Change dm_report_field_string() API to accept const char *const *data.
 
 Version 1.02.63 - 9th February 2011
 ===================================

diff --git a/libdm/libdevmapper.h b/libdm/libdevmapper.h
index 297b2c675b857105a888d0b753f3e9ce1aea6dbc..ac403e8b007558abe3cc191bca79d41b251e93db 100644 (file)
--- a/libdm/libdevmapper.h
+++ b/libdm/libdevmapper.h
@@ -1123,7 +1123,7 @@ int dm_report_set_output_field_name_prefix(struct dm_report *rh,
  * They take care of allocating copies of the data.
  */
 int dm_report_field_string(struct dm_report *rh, struct dm_report_field *field,
-                          const char **data);
+                          const char *const *data);
 int dm_report_field_int32(struct dm_report *rh, struct dm_report_field *field,
                          const int32_t *data);
 int dm_report_field_uint32(struct dm_report *rh, struct dm_report_field *field,

diff --git a/libdm/libdm-report.c b/libdm/libdm-report.c
index 4f47c46ea33093293767b767b16393d1bc81ee0f..021228414432afe26b6810d356157dd27b28b0bc 100644 (file)
--- a/libdm/libdm-report.c
+++ b/libdm/libdm-report.c
@@ -102,7 +102,7 @@ static const struct dm_report_object_type *_find_type(struct dm_report *rh,
  */
 
 int dm_report_field_string(struct dm_report *rh,
-                          struct dm_report_field *field, const char **data)
+                          struct dm_report_field *field, const char *const *data)
 {
        char *repstr;