On Fri, 05 Oct 2012 21:08:39 +0200, Diego Biurrun <[email protected]> wrote:
> ---
> libavdevice/timefilter.c | 10 +++++-----
> 1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/libavdevice/timefilter.c b/libavdevice/timefilter.c
> index 2e26b27..3a386d7 100644
> --- a/libavdevice/timefilter.c
> +++ b/libavdevice/timefilter.c
> @@ -28,8 +28,8 @@
> #include "timefilter.h"
>
> struct TimeFilter {
> - /// Delay Locked Loop data. These variables refer to mathematical
> - /// concepts described in: http://www.kokkinizita.net/papers/usingdll.pdf
> + // Delay Locked Loop data. These variables refer to mathematical
> + // concepts described in: http://www.kokkinizita.net/papers/usingdll.pdf
This isn't "normal code", it's documenting a member of a struct. Nothing
wrong with it being doxygen.
> double cycle_time;
> double feedback2_factor;
> double feedback3_factor;
> @@ -62,15 +62,15 @@ double ff_timefilter_update(TimeFilter *self, double
> system_time, double period)
> {
> self->count++;
> if (self->count == 1) {
> - /// init loop
> + // init loop
> self->cycle_time = system_time;
> } else {
> double loop_error;
> self->cycle_time += self->clock_period * period;
> - /// calculate loop error
> + // calculate loop error
> loop_error = system_time - self->cycle_time;
>
> - /// update loop
> + // update loop
> self->cycle_time += FFMAX(self->feedback2_factor, 1.0 /
> self->count) * loop_error;
> self->clock_period += self->feedback3_factor * loop_error / period;
> }
> --
> 1.7.1
Those two are ok.
--
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
