[parted-devel] [PATCH 2/2] parted: make align-check work in interactive mode
Petr Uzel
petr.uzel at suse.cz
Thu May 27 08:52:25 UTC 2010
On Wed, May 26, 2010 at 07:18:12PM +0200, Jim Meyering wrote:
> Petr Uzel wrote:
> > * parted/parted.c (do_align_check): rework the function so that
> > it reports partition alignment in interactive mode as described in
> > documentation.
> > ---
> > parted/parted.c | 21 ++++++++++++++++-----
> > 1 files changed, 16 insertions(+), 5 deletions(-)
> >
> > diff --git a/parted/parted.c b/parted/parted.c
> > index 830075b..7198b92 100644
> > --- a/parted/parted.c
> > +++ b/parted/parted.c
> > @@ -2078,18 +2078,29 @@ do_align_check (PedDevice **dev)
> > {
> > PedDisk *disk = ped_disk_new (*dev);
> > if (!disk)
> > - return 0;
> > + goto error;
> >
> > enum AlignmentType align_type = PA_OPTIMUM;
> > PedPartition *part = NULL;
> > - bool aligned =
> > - (command_line_get_align_type (_("alignment type(min/opt)"), &align_type)
> > - && command_line_get_partition (_("Partition number?"), disk, &part)
> > - && partition_align_check (disk, part, align_type));
> > +
> > + if (!command_line_get_align_type (_("alignment type(min/opt)"), &align_type))
> > + goto error_destroy_disk;
> > + if (!command_line_get_partition (_("Partition number?"), disk, &part))
> > + goto error_destroy_disk;
> > +
> > + bool aligned = partition_align_check (disk, part, align_type);
> > + if (!opt_script_mode)
> > + printf(aligned ? _("%d aligned\n") : _("%d NOT aligned\n"), part->num);
>
> Thanks. Applied modulo this:
>
> - printf(aligned ? _("%d aligned\n") : _("%d NOT aligned\n"), part->num);
> + printf(aligned ? _("%d aligned\n") : _("%d not aligned\n"), part->num);
>
> since the documentation says lower case "not".
>
> doc/parted.texi:or @samp{@var{N} not aligned}.
Ah, sure, thanks.
What about the 'FIXME:' part in original patch? It seems to work fine
as is, but perhaps I'm missing something...
Thanks,
Petr
--
Petr Uzel
IRC: ptr_uzl @ freenode
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/parted-devel/attachments/20100527/5d3f7af1/attachment.pgp>
More information about the parted-devel
mailing list