[parted-devel] policy regarding unused functions?

David Cantrell dcantrell at redhat.com
Thu Feb 15 17:52:17 CET 2007


On Thu, 2007-02-15 at 17:48 +0100, Jim Meyering wrote:
> Otavio Salvador <otavio at debian.org> wrote:
> > Jim Meyering <jim at meyering.net> writes:
> >
> >> Hello,
> >>
> >> I'm preparing a patch that makes the build warning free,
> >> and found some unused functions in libparted/labels/fdasd.c.
> >> Here they are:
> >>
> >> 	(fdasd_partition_type):
> >> 	(fdasd_write_vtoc_labels):
> >> 	(fdasd_change_volser):
> >> 	(fdasd_check_volser):
> >> 	(fdasd_reread_partition_table):
> >
> > On this specific case I think that would be better to leave fdasd out
> > of your patch. We have a fix on Debian package that needs to be ported
> > (from 1.7 release) and then dasd could be cleaned up.
> 
> I'm easy.
> But you say "fdasd", then "dasd".  I presume the latter should be "fdasd".

We say 'dasd' when we refer to zSeries disk support in general.  The
actual source files are fdasd, vtoc, something else probably.

> However, are you worried about the risk of a merge conflict?
> Unless it's more than a strategic fix, they shouldn't be changing dead code

I think the concern is more about possibly needing those unused
functions in the Hercules environment, but not on real zSeries hardware.
We won't know until I or someone else sits down and figures out parted
on hercules.

-- 
David Cantrell <dcantrell at redhat.com>
Red Hat / Westford, MA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.alioth.debian.org/pipermail/parted-devel/attachments/20070215/d2145840/attachment.pgp


More information about the parted-devel mailing list