returning a reference to the Xact

ledger-rs · Aug 8, 2023 · 1c8b460 · 1c8b460
1 parent cb29462
commit 1c8b460
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/journal.rs b/src/journal.rs
@@ -52,9 +52,10 @@ impl Journal {
         self.accounts.len() - 1
     }
 
-    pub fn add_xact(&mut self, xact: Xact) -> XactIndex {
+    pub fn add_xact(&mut self, xact: Xact) -> &Xact {
         self.xacts.push(xact);
-        self.xacts.len() - 1
+        //self.xacts.len() - 1
+        self.xacts.last().unwrap()
     }
 
     pub fn all_posts(&self) -> Vec<&Post> {

diff --git a/src/parser.rs b/src/parser.rs
@@ -156,7 +156,7 @@ impl<'j, T: Read> Parser<'j, T> {
 
             '0' | '1' | '2' | '3' | '4' | '5' | '6' | '7' | '8' | '9' => {
                 // Starts with date/number.
-                self.xact_directive();
+                let _ = self.xact_directive();
             }
 
             ' ' | '\t' => {
@@ -248,11 +248,10 @@ impl<'j, T: Read> Parser<'j, T> {
         let tokens = scanner::tokenize_xact_header(&self.buffer);
         let xact = Xact::create(tokens[0], tokens[1], tokens[2], tokens[3]);
 
-        let ptr = &xact as *const Xact;
         // Add xact to the journal
-        self.journal.add_xact(xact);
+        let new_ref = self.journal.add_xact(xact);
 
-        ptr
+        new_ref
     }
 
     fn xact_directive(&mut self) -> Result<(), Error> {