From c5775b689720c811905ebf55ee07f16773126215 Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Tue, 22 Jan 2013 13:36:53 +0100 Subject: [PATCH] affile: Restore fsync & flush-lines functionality Back in May, in 101cb379, which moved fsync() from logtransport to logproto, a new argument for log_proto_file_writer_new() was introduced, but when updating the call sites, the flush_lines and fsync arguments were mixed up. This corrects that, restoring both flush-lines() and fsync() functionality for the file destination. Reported-by: Evan Rempel Signed-off-by: Gergely Nagy --- modules/affile/affile-dest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/affile/affile-dest.c b/modules/affile/affile-dest.c index 98adee05..2df19afc 100644 --- a/modules/affile/affile-dest.c +++ b/modules/affile/affile-dest.c @@ -172,8 +172,8 @@ affile_dw_reopen(AFFileDestWriter *self) self->owner->flags & AFFILE_PIPE ? log_proto_text_client_new(log_transport_pipe_new(fd), &self->owner->writer_options.proto_options.super) : log_proto_file_writer_new(log_transport_file_new(fd), &self->owner->writer_options.proto_options.super, - self->owner->writer_options.flush_lines, - (self->owner->flags & AFFILE_FSYNC))); + (self->owner->flags & AFFILE_FSYNC), + self->owner->writer_options.flush_lines)); main_loop_call((void * (*)(void *)) affile_dw_arm_reaper, self, TRUE); }