Re: [PATCH 02/12] media: API to get/set ISO9660 volume size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 16, 2015 at 01:42:46PM +0000, Zeeshan Ali (Khattak) wrote:
> On Mon, Mar 16, 2015 at 9:56 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > On Sun, Mar 15, 2015 at 04:04:49PM +0000, Zeeshan Ali (Khattak) wrote:
> >> ---
> >>  osinfo/libosinfo.syms |  5 +++++
> >>  osinfo/osinfo_media.c | 43 +++++++++++++++++++++++++++++++++++++++++++
> >>  osinfo/osinfo_media.h |  2 ++
> >>  3 files changed, 50 insertions(+)
> >>
> >> diff --git a/osinfo/libosinfo.syms b/osinfo/libosinfo.syms
> >> index 5c423b9..2dba411 100644
> >> --- a/osinfo/libosinfo.syms
> >> +++ b/osinfo/libosinfo.syms
> >> @@ -505,6 +505,11 @@ LIBOSINFO_0.2.11 {
> >>       osinfo_install_script_get_needs_internet;
> >>  } LIBOSINFO_0.2.10;
> >>
> >> +LIBOSINFO_0.2.12 {
> >> +    global:
> >> +     osinfo_media_get_volume_size;
> >> +} LIBOSINFO_0.2.11;
> >> +
> >>  /* Symbols in next release...
> >>
> >>    LIBOSINFO_0.0.2 {
> >> diff --git a/osinfo/osinfo_media.c b/osinfo/osinfo_media.c
> >> index 96f9603..e86df11 100644
> >> --- a/osinfo/osinfo_media.c
> >> +++ b/osinfo/osinfo_media.c
> >> @@ -156,6 +156,7 @@ enum {
> >>      PROP_INSTALLER_REBOOTS,
> >>      PROP_OS,
> >>      PROP_LANGUAGES,
> >> +    PROP_VOLUME_SIZE
> >>  };
> >>
> >>  static void
> >> @@ -230,6 +231,11 @@ osinfo_media_get_property(GObject    *object,
> >>          g_value_set_pointer(value, osinfo_media_get_languages(media));
> >>          break;
> >>
> >> +    case PROP_VOLUME_SIZE:
> >> +        g_value_set_int64(value,
> >> +                          osinfo_media_get_volume_size(media));
> >> +        break;
> >> +
> >>      default:
> >>          /* We don't have any other property... */
> >>          G_OBJECT_WARN_INVALID_PROPERTY_ID(object, property_id, pspec);
> >> @@ -320,6 +326,12 @@ osinfo_media_set_property(GObject      *object,
> >>          osinfo_media_set_languages(media, g_value_get_pointer(value));
> >>          break;
> >>
> >> +    case PROP_VOLUME_SIZE:
> >> +        osinfo_entity_set_param_int64(OSINFO_ENTITY(media),
> >> +                                      OSINFO_MEDIA_PROP_VOLUME_SIZE,
> >> +                                      g_value_get_int64(value));
> >> +        break;
> >> +
> >>      default:
> >>          /* We don't have any other property... */
> >>          G_OBJECT_WARN_INVALID_PROPERTY_ID(object, property_id, pspec);
> >> @@ -547,6 +559,21 @@ osinfo_media_class_init(OsinfoMediaClass *klass)
> >>                                   G_PARAM_READABLE |
> >>                                   G_PARAM_STATIC_STRINGS);
> >>      g_object_class_install_property(g_klass, PROP_LANGUAGES, pspec);
> >> +
> >> +    /**
> >> +     * OsinfoMedia:volume-size:
> >> +     *
> >> +     * Expected volume size, in bytes for ISO9660 image/device.
> >> +     */
> >> +    pspec = g_param_spec_int64("volume-size",
> >> +                               "VolumeSize",
> >
> > As this field does not exist as is in the ISO9660 spec, I'd use a more
> > usual spelling for "Volume Size" here.
> 
> I'm just following the convention we seem to be following for other
> properties' nick, nothing to do with ISO9660 spec..

I swear when I looked this morning, only the properties directly
corresponding to ISO9660 fields were using sturdy caps ;) Now that I
look again, it's indeed done this way for all nicks in this file, sorry
for the noise.

Christophe

Attachment: pgpCfrbe5VpuZ.pgp
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux