From 8863baacd9fb25a4d70c659149de1a2e78c52953 Mon Sep 17 00:00:00 2001 From: Brian Buller Date: Fri, 22 Feb 2019 17:29:59 -0600 Subject: [PATCH] Use pointers to task lists --- todotxt.go | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/todotxt.go b/todotxt.go index bc199c8..4f60f22 100644 --- a/todotxt.go +++ b/todotxt.go @@ -30,9 +30,8 @@ var ( ) // NewTaskList creates a new empty TaskList. -func NewTaskList() TaskList { - tasklist := TaskList{} - return tasklist +func NewTaskList() *TaskList { + return &TaskList{} } // String returns a complete list of tasks in todo.txt format. @@ -109,6 +108,9 @@ func (tasklist *TaskList) RemoveTask(task Task) error { return nil } +// ArchiveTaskToFile removes the task from the active list and concatenates it to +// the passed in filename +// Return an err if any part of that fails func (tasklist *TaskList) ArchiveTaskToFile(task Task, filename string) error { if err := tasklist.RemoveTask(task); err != nil { return err @@ -118,7 +120,7 @@ func (tasklist *TaskList) ArchiveTaskToFile(task Task, filename string) error { return err } defer f.Close() - _, err = f.WriteString("\n" + task.String()) + _, err = f.WriteString(task.String() + "\n") return err }