On 2015-08-14 17:35 -0400, Ganesh Ajjanagadde wrote:
> On Fri, Aug 14, 2015 at 4:46 PM, Alexander Strasser <eclip...@gmx.net> wrote:
> > Signed-off-by: Alexander Strasser <eclip...@gmx.net>
> > ---
> >  libavutil/parseutils.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavutil/parseutils.h b/libavutil/parseutils.h
> > index b04aa89..cff5123 100644
> > --- a/libavutil/parseutils.h
> > +++ b/libavutil/parseutils.h
> > @@ -112,7 +112,7 @@ int av_parse_color(uint8_t *rgba_color, const char 
> > *color_string, int slen,
> >  const char *av_get_known_color_name(int color_idx, const uint8_t **rgb);
> >
> >  /**
> > - * Parse timestr and return in *timeval a corresponding number of
> > + * Parse timestr and return in *timeval the corresponding number of
> >   * microseconds.
> >   *
> >   * @param timeval puts here the number of microseconds corresponding
> > --
> >
> So how exactly is this different from patch 1/3?
> Are you sure you emailed the right patch?

Here I replaced "a" with "the". Saying it returns *a* corresponding
number of microseconds kind of suggests there is more then one value
it could choose from.

Well maybe you could say the original is kind of true for dates
because of timezones etc...  Anyway it is more like a cosmetic
change if it doesn't resonate with others, I will just drop
this patch.


  Alexander

Attachment: pgp7DuWyGksbU.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to