qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alon Levy <alevy@redhat.com>
To: Stefan Weil <sw@weilnetz.de>
Cc: QEMU Trivial <qemu-trivial@nongnu.org>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] libcacard: Spelling and grammar fixes in documentation
Date: Sun, 26 Feb 2012 14:38:12 +0100	[thread overview]
Message-ID: <20120226133812.GC7395@garlic.redhat.com> (raw)
In-Reply-To: <1330263021-1339-1-git-send-email-sw@weilnetz.de>

On Sun, Feb 26, 2012 at 02:30:21PM +0100, Stefan Weil wrote:

Ack.

> * it's -> its
> 
> * it's -> it is (that's no fix, but makes future checks easier)
> 
> * this functions -> this function
> 
> * replacable -> replaceable
> 
> * reader's -> readers
> 
> * logins into -> logs into
> 
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> ---
>  docs/libcacard.txt |   22 +++++++++++-----------
>  1 files changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/docs/libcacard.txt b/docs/libcacard.txt
> index f7d7519..559db8a 100644
> --- a/docs/libcacard.txt
> +++ b/docs/libcacard.txt
> @@ -10,7 +10,7 @@ such as signing, card removal/insertion, etc. are mapped to real, physical
>  cards which are shared with the client machine the emulator is running on, or
>  the cards could be pure software constructs.
>  
> -The emulator is structured to allow multiple replacable or additional pieces,
> +The emulator is structured to allow multiple replaceable or additional pieces,
>  so it can be easily modified for future requirements. The primary envisioned
>  modifications are:
>  
> @@ -32,7 +32,7 @@ be emulated as well, including PIV, newer versions of CAC, PKCS #15, etc.
>  --------------------
>  Replacing the Socket Based Virtual Reader Interface.
>  
> -The current implementation contains a replacable module vscclient.c. The
> +The current implementation contains a replaceable module vscclient.c. The
>  current vscclient.c implements a sockets interface to the virtual ccid reader
>  on the guest. CCID commands that are pertinent to emulation are passed
>  across the socket, and their responses are passed back along that same socket.
> @@ -42,7 +42,7 @@ implements a program with a main entry. It also handles argument parsing for
>  the emulator.
>  
>  An application that wants to use the virtual reader can replace vscclient.c
> -with it's own implementation that connects to it's own CCID reader.  The calls
> +with its own implementation that connects to its own CCID reader.  The calls
>  that the CCID reader can call are:
>  
>        VReaderList * vreader_get_reader_list();
> @@ -72,12 +72,12 @@ that the CCID reader can call are:
>        VReader * vreader_list_get_reader(VReaderListEntry *)
>  
>    This function returns the reader stored in the reader List entry. Caller gets
> -  a new reference to a reader. The caller must free it's reference when it is
> +  a new reference to a reader. The caller must free its reference when it is
>    finished with vreader_free().
>  
>        void vreader_free(VReader *reader);
>  
> -   This function frees a reference to a reader. Reader's are reference counted
> +   This function frees a reference to a reader. Readers are reference counted
>     and are automatically deleted when the last reference is freed.
>  
>        void vreader_list_delete(VReaderList *list);
> @@ -87,7 +87,7 @@ that the CCID reader can call are:
>  
>        VReaderStatus vreader_power_on(VReader *reader, char *atr, int *len);
>  
> -  This functions simulates a card power on. Virtual cards do not care about
> +  This function simulates a card power on. A virtual card does not care about
>    the actual voltage and other physical parameters, but it does care that the
>    card is actually on or off. Cycling the card causes the card to reset. If
>    the caller provides enough space, vreader_power_on will return the ATR of
> @@ -104,7 +104,7 @@ that the CCID reader can call are:
>                                         unsigned char *receive_buf,
>                                         int receive_buf_len);
>  
> -  This functions send a raw apdu to a card and returns the card's response.
> +  This function sends a raw apdu to a card and returns the card's response.
>    The CCID front end should return the response back. Most of the emulation
>    is driven from these APDUs.
>  
> @@ -217,10 +217,10 @@ the card using the following functions:
>           VCardStatus vcard_add_applet(VCard *card, VCardApplet *applet);
>  
>    Add an applet onto the list of applets attached to the card. Once an applet
> -  has been added, it can be selected by it's aid, and then commands will be
> +  has been added, it can be selected by its aid, and then commands will be
>    routed to it VCardProcessAPDU function. This function adopts the applet the
>    passed int applet. Note: 2 applets with the same AID should not be added to
> -  the same card. It's permissible to add more than one applet. Multiple applets
> +  the same card. It is permissible to add more than one applet. Multiple applets
>    may have the same VCardPRocessAPDU entry point.
>  
>  The certs and keys should be attached to private data associated with one or
> @@ -335,7 +335,7 @@ and applet.
>           VCard7816Status vcard_emul_login(VCard *card, unsigned char *pin,
>                                            int pin_len);
>  
> -    This function logins into the card and return the standard 7816 status
> +    This function logs into the card and returns the standard 7816 status
>      word depending on the success or failure of the call.
>  
>           void vcard_emul_delete_key(VCardKey *key);
> @@ -424,7 +424,7 @@ functions:
>        cert_len, and keys are all arrays of length cert_count. These are the
>        the same of the parameters xxxx_card_init() accepts.
>  
> -   Finally the card is associated with it's reader by the call:
> +   Finally the card is associated with its reader by the call:
>  
>              VReaderStatus vreader_insert_card(VReader *vreader, VCard *vcard);
>  
> -- 
> 1.7.9
> 
> 

  reply	other threads:[~2012-02-26 13:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-26 13:30 [Qemu-devel] [PATCH] libcacard: Spelling and grammar fixes in documentation Stefan Weil
2012-02-26 13:38 ` Alon Levy [this message]
2012-02-26 18:18 ` Peter Maydell
2012-02-26 18:52   ` Stefan Weil
2012-02-27 21:24   ` Alon Levy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120226133812.GC7395@garlic.redhat.com \
    --to=alevy@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-trivial@nongnu.org \
    --cc=sw@weilnetz.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).