From 42c693f70ff375ce15b5783c251bfa723ac528ce Mon Sep 17 00:00:00 2001
From: Dominik Kaiser
Date: Mon, 1 Jul 2024 11:44:06 +0200
Subject: Remove unneeded struct s_sequence

---
 include/ast.h | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

(limited to 'include/ast.h')

diff --git a/include/ast.h b/include/ast.h
index e6ad25d..356ccdd 100644
--- a/include/ast.h
+++ b/include/ast.h
@@ -6,18 +6,13 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:48:27 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/06/28 14:56:55 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/01 11:43:59 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
 #include "stdlib.h"
 #include "debug_tools.h"
 
-typedef struct s_sequence
-{
-	struct s_node			**nodes;
-}							t_sequence;
-
 enum						e_node_type
 {
 	ASSIGN_NODE,
-- 
cgit v1.2.3


From 5316c9880416b77b4b97b07fd6ae47f171a0ba23 Mon Sep 17 00:00:00 2001
From: Dominik Kaiser
Date: Mon, 8 Jul 2024 16:15:32 +0200
Subject: Move assign into cmd node

---
 include/ast.h   |  8 +++-----
 src/free_node.c | 24 +++++++++++++++++++-----
 src/new_node.c  | 17 +++--------------
 3 files changed, 25 insertions(+), 24 deletions(-)

(limited to 'include/ast.h')

diff --git a/include/ast.h b/include/ast.h
index 356ccdd..6479272 100644
--- a/include/ast.h
+++ b/include/ast.h
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:48:27 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/01 11:43:59 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/08 16:09:32 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -15,7 +15,6 @@
 
 enum						e_node_type
 {
-	ASSIGN_NODE,
 	PIPE_NODE,
 	CMD_NODE,
 	STRING_NODE
@@ -50,12 +49,12 @@ typedef struct s_redirection
 typedef struct s_cmd
 {
 	char					**args;
+	struct s_assign			**assigns;
 	struct s_redirection	redirs[2];
 }							t_cmd;
 
 union						u_node_content
 {
-	struct s_assign			assign;
 	struct s_pipe			pipe;
 	struct s_cmd			cmd;
 	char					*string;
@@ -68,7 +67,6 @@ typedef struct s_node
 }							t_node;
 
 t_node						*new_node(int type);
-t_node						*new_assign_node(char *var, char *value);
 t_node						*new_pipe_node(t_node *left, t_node *right);
-t_node						*new_cmd_node(char **args, t_redirection redirs[2]);
+t_node						*new_cmd_node(char **args, t_assign  **assigns, t_redirection redirs[2]);
 t_node						*new_string_node(char *string);
diff --git a/src/free_node.c b/src/free_node.c
index 8f32c12..8664a07 100644
--- a/src/free_node.c
+++ b/src/free_node.c
@@ -6,21 +6,19 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:41:46 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/06/28 14:55:50 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/08 16:15:21 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
 #include "ast.h"
 
-static void	free_assign_node(t_node *node);
 static void	free_pipe_node(t_node *node);
 static void	free_cmd_node(t_node *node);
+static void	free_assigns(t_assign **assigns);
 
 void	free_node(t_node *node)
 {
-	if (node->type == ASSIGN_NODE)
-		free_assign_node(node);
-	else if (node->type == PIPE_NODE)
+	if (node->type == PIPE_NODE)
 		free_pipe_node(node);
 	else if (node->type == CMD_NODE)
 		free_cmd_node(node);
@@ -54,6 +52,22 @@ static void	free_cmd_node(t_node *node)
 		i++;
 	}
 	free(node->content.cmd.args);
+	free_assigns(node->content.cmd.assigns);
 	free(node->content.cmd.redirs[0].specifier);
 	free(node->content.cmd.redirs[1].specifier);
 }
+
+static void	free_assigns(t_assign **assigns)
+{
+	int	i;
+
+	i = 0;
+	while (assigns[i] != NULL)
+	{
+		free(assigns[i]->var);
+		free(assigns[i]->value);
+		free(assigns[i]);
+		i++;
+	}
+	free(assigns);
+}
diff --git a/src/new_node.c b/src/new_node.c
index 4cdbf9a..c334eb6 100644
--- a/src/new_node.c
+++ b/src/new_node.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:21:03 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/06/28 15:04:15 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/08 16:10:44 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -23,18 +23,6 @@ t_node	*new_node(int type)
 	return (node);
 }
 
-t_node	*new_assign_node(char *var, char *value)
-{
-	t_node	*node;
-
-	node = new_node(ASSIGN_NODE);
-	if (node == NULL)
-		return (NULL);
-	node->content.assign.var = var;
-	node->content.assign.value = value;
-	return (node);
-}
-
 t_node	*new_pipe_node(t_node *left, t_node *right)
 {
 	t_node	*node;
@@ -47,7 +35,7 @@ t_node	*new_pipe_node(t_node *left, t_node *right)
 	return (node);
 }
 
-t_node	*new_cmd_node(char **args, t_redirection redirs[2])
+t_node	*new_cmd_node(char **args, t_assign **assigns, t_redirection redirs[2])
 {
 	t_node	*node;
 
@@ -55,6 +43,7 @@ t_node	*new_cmd_node(char **args, t_redirection redirs[2])
 	if (node == NULL)
 		return (NULL);
 	node->content.cmd.args = args;
+	node->content.cmd.assigns = assigns;
 	node->content.cmd.redirs[0] = redirs[0];
 	node->content.cmd.redirs[1] = redirs[1];
 	return (node);
-- 
cgit v1.2.3


From 8f386bcd29425b96017c1e9b2cd1bbd670939563 Mon Sep 17 00:00:00 2001
From: Dominik Kaiser
Date: Tue, 9 Jul 2024 19:23:17 +0200
Subject: Adding working parsing

TODO: Refactoring
---
 include/ast.h    |  10 ++---
 include/token.h  |  10 ++---
 src/free_token.c |   4 +-
 src/parse_cmd.c  | 122 +++++++++++++++++++++++++++++++++++++++++--------------
 4 files changed, 102 insertions(+), 44 deletions(-)

(limited to 'include/ast.h')

diff --git a/include/ast.h b/include/ast.h
index 6479272..b0ed996 100644
--- a/include/ast.h
+++ b/include/ast.h
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:48:27 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/08 16:09:32 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/09 16:54:29 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -34,10 +34,10 @@ typedef struct s_pipe
 
 enum						e_redirection_type
 {
-	INPUT_FILE,
-	INPUT_LIMITER,
-	OUTPUT_OVERRIDE,
-	OUTPUT_APPEND
+	INPUT_FILE = 1,
+	INPUT_LIMITER = 2,
+	OUTPUT_OVERRIDE = 4,
+	OUTPUT_APPEND = 8
 };
 
 typedef struct s_redirection
diff --git a/include/token.h b/include/token.h
index 32b042d..d61195d 100644
--- a/include/token.h
+++ b/include/token.h
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 13:27:18 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/08 14:48:57 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/09 18:38:50 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -17,10 +17,10 @@
 
 enum						e_token_type
 {
-	STRING_TOKEN,
-	PIPE_TOKEN,
-	REDIR_TOKEN,
-	NEWLINE_TOKEN
+	STRING_TOKEN = 1,
+	PIPE_TOKEN = 2,
+	REDIR_TOKEN = 4,
+	NEWLINE_TOKEN = 8
 };
 
 union						u_token_content
diff --git a/src/free_token.c b/src/free_token.c
index 083484c..a7b1ef2 100644
--- a/src/free_token.c
+++ b/src/free_token.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 14:38:57 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/08 16:44:38 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/09 14:30:02 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -23,8 +23,6 @@ void	free_token(t_token *token)
 
 void	free_token_and_connect(t_token *token)
 {
-	if (token->type == STRING_TOKEN)
-		free(token->content.string);
 	if (token->previous != NULL)
 		token->previous->next = token->next;
 	if (token->next != NULL)
diff --git a/src/parse_cmd.c b/src/parse_cmd.c
index 739c022..4fa2bc2 100644
--- a/src/parse_cmd.c
+++ b/src/parse_cmd.c
@@ -6,17 +6,13 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/07/08 15:06:25 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/08 17:29:22 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/09 19:22:51 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
-#include "libft.h"
 #include "minishell.h"
-#include "token.h"
-#include <stdlib.h>
-#include <string.h>
 
-static t_redirection	**collect_redirs(t_token **tokens);
+static t_redirection	*collect_redirs(t_token **tokens);
 static t_assign			**collect_assigns(t_token **tokens);
 static char				**collect_args(t_token **tokens);
 static t_assign			*to_assign(char *str);
@@ -25,7 +21,7 @@ t_node	*parse_cmd(t_token *tokens)
 {
 	char			**args;
 	t_assign		**assigns;
-	t_redirection	redirs[2];
+	t_redirection	*redirs;
 
 	redirs = collect_redirs(&tokens);
 	assigns = collect_assigns(&tokens);
@@ -33,80 +29,144 @@ t_node	*parse_cmd(t_token *tokens)
 	return (new_cmd_node(args, assigns, redirs));
 }
 
-static t_redirection	**collect_redirs(t_token **tokens)
+static t_redirection	*collect_redirs(t_token **tokens)
 {
-	return (NULL);
+	t_redirection	*result;
+	t_token			*cur;
+
+	cur = *tokens;
+	result = malloc(sizeof(t_redirection) * 2);
+	if (result == NULL)
+	{
+		// free all tokens
+		return (NULL);
+	}
+	result[0].specifier = NULL;
+	result[0].type = 0;
+	result[1].specifier = NULL;
+	result[1].type = 0;
+	while (cur != NULL && cur->next != NULL)
+	{
+		if (cur->type == REDIR_TOKEN && cur->next->type == STRING_TOKEN)
+		{
+			if (cur->content.redir_type & (INPUT_FILE | INPUT_LIMITER))
+			{
+				result[0].type = cur->content.redir_type;
+				result[0].specifier = cur->next->content.string;
+			}
+			else if (cur->content.redir_type & (OUTPUT_APPEND | OUTPUT_OVERRIDE))
+			{
+				result[1].type = cur->content.redir_type;
+				result[1].specifier = cur->next->content.string;
+			}
+			cur = cur->next;
+			free_token_and_connect(cur->previous);
+			if (cur->next != NULL)
+			{
+				cur = cur->next;
+				free_token_and_connect(cur->previous);
+			}
+			else
+			{
+				free_token(cur);
+			}
+		}
+		else if (cur->type == REDIR_TOKEN)
+		{
+			// err Parsing error
+			free(result);
+			return (NULL);
+		}
+		else
+		{
+			cur = cur->next;
+		}
+	}
+	return (result);
 }
 
 static t_assign	**collect_assigns(t_token **tokens)
 {
-	int			i;
+	t_token		*cur;
 	t_assign	**result;
+	int			i;
 
+	cur = *tokens;
 	i = 0;
-	while (ft_strchr(tokens[i]->content.string, '=') != NULL)
+	while (cur != NULL && cur->type == STRING_TOKEN
+		&& cur->content.string[0] != '"' && cur->content.string[0] != '\''
+		&& ft_strchr(cur->content.string, '=') != NULL)
 	{
 		i++;
+		cur = cur->next;
 	}
 	result = malloc(sizeof(t_assign *) * (i + 1));
-	if (result == NULL)
+	if (!result)
 	{
-		// free everything
+		// free all tokens
 		return (NULL);
 	}
-	result[i] = NULL;
-	i--;
-	while (i >= 0)
+	cur = *tokens;
+	i = 0;
+	while (cur != NULL && cur->type == STRING_TOKEN
+		&& cur->content.string[0] != '"' && cur->content.string[0] != '\''
+		&& ft_strchr(cur->content.string, '=') != NULL)
 	{
-		result[i] = to_assign(tokens[i]->content.string);
-		i--;
+		result[i] = to_assign(cur->content.string);
+		i++;
+		cur = cur->next;
+		free_token(cur->previous);
 	}
+	*tokens = cur;
+	result[i] = NULL;
 	return (result);
 }
 
 static t_assign	*to_assign(char *str)
 {
 	t_assign	*result;
-	char		*split;
+	char		*split_pos;
 
+	split_pos = ft_strchr(str, '=');
+	*split_pos = '\0';
 	result = malloc(sizeof(t_assign));
 	if (result == NULL)
+	{
 		return (NULL);
-	split = ft_strchr(str, '=');
-	if (split == NULL)
-		return (NULL);
-	*split = '\0';
+	}
 	result->var = str;
-	result->value = split + 1;
+	result->value = split_pos + 1;
 	return (result);
 }
 
 static char	**collect_args(t_token **tokens)
 {
-	t_token *cur;
-	char **result;
-	int i;
+	t_token	*cur;
+	char	**result;
+	int		i;
 
 	cur = *tokens;
 	i = 0;
-	while (cur != NULL) {
+	while (cur != NULL)
+	{
 		i++;
 		cur = cur->next;
 	}
-	result = malloc(sizeof(char*) * (i + 1));
+	result = malloc(sizeof(char *) * (i + 1));
 	if (!result)
 	{
-		//free all tokens;
+		// free all tokens;
 		return (NULL);
 	}
 	cur = *tokens;
 	i = 0;
-	while(cur != NULL)
+	while (cur != NULL)
 	{
 		result[i] = cur->content.string;
 		// free token
 		i++;
 		cur = cur->next;
 	}
+	result[i] = NULL;
 	return (result);
 }
-- 
cgit v1.2.3


From 47a68b82e0463f5dfd4bd8a9c4b32b4c5fbcb610 Mon Sep 17 00:00:00 2001
From: Dominik Kaiser
Date: Mon, 22 Jul 2024 15:07:59 +0200
Subject: Fix errors and make the parser work

---
 include/ast.h   |  9 ++++++---
 src/free_node.c | 13 ++++++++-----
 src/new_node.c  |  3 ++-
 src/parse_cmd.c | 29 ++++++++++++++++-------------
 4 files changed, 32 insertions(+), 22 deletions(-)

(limited to 'include/ast.h')

diff --git a/include/ast.h b/include/ast.h
index b0ed996..bf19083 100644
--- a/include/ast.h
+++ b/include/ast.h
@@ -6,12 +6,12 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:48:27 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/09 16:54:29 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/10 12:31:39 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
-#include "stdlib.h"
 #include "debug_tools.h"
+#include "stdlib.h"
 
 enum						e_node_type
 {
@@ -68,5 +68,8 @@ typedef struct s_node
 
 t_node						*new_node(int type);
 t_node						*new_pipe_node(t_node *left, t_node *right);
-t_node						*new_cmd_node(char **args, t_assign  **assigns, t_redirection redirs[2]);
+t_node						*new_cmd_node(char **args, t_assign **assigns,
+								t_redirection redirs[2]);
 t_node						*new_string_node(char *string);
+
+void						free_node(t_node *node);
diff --git a/src/free_node.c b/src/free_node.c
index 2dfeb7d..f8677d5 100644
--- a/src/free_node.c
+++ b/src/free_node.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:41:46 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/09 11:37:11 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/22 14:30:14 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -40,15 +40,17 @@ static void	free_cmd_node(t_node *node)
 	int	i;
 
 	i = 0;
-	while (node->content.cmd.args[i] != NULL)
+	while (node->content.cmd.args != NULL && node->content.cmd.args[i] != NULL)
 	{
 		free(node->content.cmd.args[i]);
 		i++;
 	}
 	free(node->content.cmd.args);
 	free_assigns(node->content.cmd.assigns);
-	free(node->content.cmd.redirs[0].specifier);
-	free(node->content.cmd.redirs[1].specifier);
+	if (node->content.cmd.redirs[0].type != 0 && node->content.cmd.redirs[0].specifier != NULL)
+		free(node->content.cmd.redirs[0].specifier);
+	if (node->content.cmd.redirs[1].type != 0 && node->content.cmd.redirs[0].specifier != NULL)
+		free(node->content.cmd.redirs[1].specifier);
 }
 
 static void	free_assigns(t_assign **assigns)
@@ -56,10 +58,11 @@ static void	free_assigns(t_assign **assigns)
 	int	i;
 
 	i = 0;
+	if (assigns == 0)
+		return ;
 	while (assigns[i] != NULL)
 	{
 		free(assigns[i]->var);
-		free(assigns[i]->value);
 		free(assigns[i]);
 		i++;
 	}
diff --git a/src/new_node.c b/src/new_node.c
index c334eb6..c2458d0 100644
--- a/src/new_node.c
+++ b/src/new_node.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:21:03 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/08 16:10:44 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/22 14:25:03 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -46,6 +46,7 @@ t_node	*new_cmd_node(char **args, t_assign **assigns, t_redirection redirs[2])
 	node->content.cmd.assigns = assigns;
 	node->content.cmd.redirs[0] = redirs[0];
 	node->content.cmd.redirs[1] = redirs[1];
+	free(redirs);
 	return (node);
 }
 
diff --git a/src/parse_cmd.c b/src/parse_cmd.c
index 4fa2bc2..0173caa 100644
--- a/src/parse_cmd.c
+++ b/src/parse_cmd.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/07/08 15:06:25 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/09 19:22:51 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/07/10 13:26:17 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -16,6 +16,8 @@ static t_redirection	*collect_redirs(t_token **tokens);
 static t_assign			**collect_assigns(t_token **tokens);
 static char				**collect_args(t_token **tokens);
 static t_assign			*to_assign(char *str);
+static void				set_redir(t_redirection *redir, int type,
+							char *specifier);
 
 t_node	*parse_cmd(t_token *tokens)
 {
@@ -33,6 +35,7 @@ static t_redirection	*collect_redirs(t_token **tokens)
 {
 	t_redirection	*result;
 	t_token			*cur;
+	int				idx;
 
 	cur = *tokens;
 	result = malloc(sizeof(t_redirection) * 2);
@@ -41,24 +44,18 @@ static t_redirection	*collect_redirs(t_token **tokens)
 		// free all tokens
 		return (NULL);
 	}
-	result[0].specifier = NULL;
-	result[0].type = 0;
-	result[1].specifier = NULL;
-	result[1].type = 0;
+	set_redir(&result[0], 0, NULL);
+	set_redir(&result[1], 0, NULL);
 	while (cur != NULL && cur->next != NULL)
 	{
 		if (cur->type == REDIR_TOKEN && cur->next->type == STRING_TOKEN)
 		{
 			if (cur->content.redir_type & (INPUT_FILE | INPUT_LIMITER))
-			{
-				result[0].type = cur->content.redir_type;
-				result[0].specifier = cur->next->content.string;
-			}
+				idx = 0;
 			else if (cur->content.redir_type & (OUTPUT_APPEND | OUTPUT_OVERRIDE))
-			{
-				result[1].type = cur->content.redir_type;
-				result[1].specifier = cur->next->content.string;
-			}
+				idx = 1;
+			set_redir(&result[idx], cur->content.redir_type,
+				cur->next->content.string);
 			cur = cur->next;
 			free_token_and_connect(cur->previous);
 			if (cur->next != NULL)
@@ -170,3 +167,9 @@ static char	**collect_args(t_token **tokens)
 	result[i] = NULL;
 	return (result);
 }
+
+static void	set_redir(t_redirection *redir, int type, char *specifier)
+{
+	redir->type = type;
+	redir->specifier = specifier;
+}
-- 
cgit v1.2.3


From 99e8655aaf9827c7d5248c7f3d0913fcb1377cfb Mon Sep 17 00:00:00 2001
From: Dominik Kaiser
Date: Sun, 11 Aug 2024 12:27:02 +0200
Subject: Remove assigns

I found out that there's a difference between shell variables and env
variables.
We don't have to implement shell variables, so I removed all code that
handles them.
---
 Makefile              |  2 +-
 include/ast.h         | 12 ++------
 include/minishell.h   |  3 +-
 src/collect_assigns.c | 84 ---------------------------------------------------
 src/free_node.c       | 20 +-----------
 src/new_node.c        |  5 ++-
 src/parse_cmd.c       |  6 ++--
 src/print_ast.c       | 10 +-----
 8 files changed, 10 insertions(+), 132 deletions(-)
 delete mode 100644 src/collect_assigns.c

(limited to 'include/ast.h')

diff --git a/Makefile b/Makefile
index d4778bf..827c317 100644
--- a/Makefile
+++ b/Makefile
@@ -13,7 +13,7 @@ HEADERS =  -I include -I $(LIB_DIR)/libft
 VPATH   := src
 SRC     := main.c debug_tools.c init.c signal_handling.c repl.c new_token.c \
            free_token.c new_node.c free_node.c tokenizer.c parser.c \
-           parse_cmd.c collect_assigns.c collect_redirs.c print_ast.c
+           parse_cmd.c collect_redirs.c print_ast.c
 
 OBJ_DIR := _obj
 OBJ     := $(addprefix $(OBJ_DIR)/, $(SRC:%.c=%.o))
diff --git a/include/ast.h b/include/ast.h
index bf19083..cd2f9c9 100644
--- a/include/ast.h
+++ b/include/ast.h
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:48:27 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/10 12:31:39 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:20:56 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -20,12 +20,6 @@ enum						e_node_type
 	STRING_NODE
 };
 
-typedef struct s_assign
-{
-	char					*var;
-	char					*value;
-}							t_assign;
-
 typedef struct s_pipe
 {
 	struct s_node			*left;
@@ -49,7 +43,6 @@ typedef struct s_redirection
 typedef struct s_cmd
 {
 	char					**args;
-	struct s_assign			**assigns;
 	struct s_redirection	redirs[2];
 }							t_cmd;
 
@@ -68,8 +61,7 @@ typedef struct s_node
 
 t_node						*new_node(int type);
 t_node						*new_pipe_node(t_node *left, t_node *right);
-t_node						*new_cmd_node(char **args, t_assign **assigns,
-								t_redirection redirs[2]);
+t_node						*new_cmd_node(char **args, t_redirection redirs[2]);
 t_node						*new_string_node(char *string);
 
 void						free_node(t_node *node);
diff --git a/include/minishell.h b/include/minishell.h
index 4f04682..6997b15 100644
--- a/include/minishell.h
+++ b/include/minishell.h
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/22 17:14:49 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/08/11 10:59:16 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:22:07 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -34,7 +34,6 @@ void			repl(const char *prompt);
 t_list			*parse(t_token *tokens);
 t_node			*parse_cmd(t_token *tokens);
 t_redirection	*collect_redirs(t_token **tokens);
-t_assign	**collect_assigns(t_token **tokens);
 
 void			print_ast(t_node *ast);
 #endif
diff --git a/src/collect_assigns.c b/src/collect_assigns.c
deleted file mode 100644
index da43503..0000000
--- a/src/collect_assigns.c
+++ /dev/null
@@ -1,84 +0,0 @@
-/* ************************************************************************** */
-/*                                                                            */
-/*                                                        :::      ::::::::   */
-/*   collect_assigns.c                                  :+:      :+:    :+:   */
-/*                                                    +:+ +:+         +:+     */
-/*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
-/*                                                +#+#+#+#+#+   +#+           */
-/*   Created: 2024/08/02 13:54:36 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/08/02 14:37:41 by dkaiser          ###   ########.fr       */
-/*                                                                            */
-/* ************************************************************************** */
-
-#include "minishell.h"
-#include "token.h"
-
-static t_assign	*to_assign(char *str);
-static int		count_tokens(t_token *tokens);
-static int		is_quote(char c);
-
-t_assign	**collect_assigns(t_token **tokens)
-{
-	t_token		*cur;
-	t_assign	**result;
-	int			i;
-
-	result = malloc(sizeof(t_assign *) * (count_tokens(*tokens) + 1));
-	if (result == NULL)
-		return (free_tokens(*tokens), NULL);
-	cur = *tokens;
-	i = 0;
-	while (cur != NULL && cur->type == STRING_TOKEN
-		&& !is_quote(cur->content.string[0]) && ft_strchr(cur->content.string,
-			'=') != NULL)
-	{
-		result[i++] = to_assign(cur->content.string);
-		if (cur->next != NULL)
-		{
-			cur = cur->next;
-			free_token(cur->previous);
-		}
-		else
-			free_token(cur);
-	}
-	*tokens = cur;
-	result[i] = NULL;
-	return (result);
-}
-
-static t_assign	*to_assign(char *str)
-{
-	t_assign	*result;
-	char		*split_pos;
-
-	split_pos = ft_strchr(str, '=');
-	*split_pos = '\0';
-	result = malloc(sizeof(t_assign));
-	if (result == NULL)
-	{
-		return (NULL);
-	}
-	result->var = str;
-	result->value = split_pos + 1;
-	return (result);
-}
-
-static int	count_tokens(t_token *tokens)
-{
-	int	len;
-
-	len = 0;
-	while (tokens != NULL && tokens->type == STRING_TOKEN
-		&& !is_quote(tokens->content.string[0])
-		&& ft_strchr(tokens->content.string, '=') != NULL)
-	{
-		len++;
-		tokens = tokens->next;
-	}
-	return (len);
-}
-
-static int	is_quote(char c)
-{
-	return (c == '"' || c == '\'');
-}
diff --git a/src/free_node.c b/src/free_node.c
index f387c0a..6eae059 100644
--- a/src/free_node.c
+++ b/src/free_node.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:41:46 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/08/02 13:28:47 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:26:20 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -14,7 +14,6 @@
 
 static void	free_pipe_node(t_node *node);
 static void	free_cmd_node(t_node *node);
-static void	free_assigns(t_assign **assigns);
 
 void	free_node(t_node *node)
 {
@@ -46,7 +45,6 @@ static void	free_cmd_node(t_node *node)
 		i++;
 	}
 	free(node->content.cmd.args);
-	free_assigns(node->content.cmd.assigns);
 	if (node->content.cmd.redirs[0].type != 0
 		&& node->content.cmd.redirs[0].specifier != NULL)
 		free(node->content.cmd.redirs[0].specifier);
@@ -54,19 +52,3 @@ static void	free_cmd_node(t_node *node)
 		&& node->content.cmd.redirs[0].specifier != NULL)
 		free(node->content.cmd.redirs[1].specifier);
 }
-
-static void	free_assigns(t_assign **assigns)
-{
-	int	i;
-
-	i = 0;
-	if (assigns == 0)
-		return ;
-	while (assigns[i] != NULL)
-	{
-		free(assigns[i]->var);
-		free(assigns[i]);
-		i++;
-	}
-	free(assigns);
-}
diff --git a/src/new_node.c b/src/new_node.c
index c2458d0..6da9f9e 100644
--- a/src/new_node.c
+++ b/src/new_node.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/06/27 11:21:03 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/07/22 14:25:03 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:22:45 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -35,7 +35,7 @@ t_node	*new_pipe_node(t_node *left, t_node *right)
 	return (node);
 }
 
-t_node	*new_cmd_node(char **args, t_assign **assigns, t_redirection redirs[2])
+t_node	*new_cmd_node(char **args, t_redirection redirs[2])
 {
 	t_node	*node;
 
@@ -43,7 +43,6 @@ t_node	*new_cmd_node(char **args, t_assign **assigns, t_redirection redirs[2])
 	if (node == NULL)
 		return (NULL);
 	node->content.cmd.args = args;
-	node->content.cmd.assigns = assigns;
 	node->content.cmd.redirs[0] = redirs[0];
 	node->content.cmd.redirs[1] = redirs[1];
 	free(redirs);
diff --git a/src/parse_cmd.c b/src/parse_cmd.c
index eb70f0d..2755cae 100644
--- a/src/parse_cmd.c
+++ b/src/parse_cmd.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/07/08 15:06:25 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/08/11 12:14:11 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:20:06 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -17,13 +17,11 @@ static char	**collect_args(t_token **tokens);
 t_node	*parse_cmd(t_token *tokens)
 {
 	char			**args;
-	t_assign		**assigns;
 	t_redirection	*redirs;
 
 	redirs = collect_redirs(&tokens);
-	assigns = collect_assigns(&tokens);
 	args = collect_args(&tokens);
-	return (new_cmd_node(args, assigns, redirs));
+	return (new_cmd_node(args, redirs));
 }
 
 static char	**collect_args(t_token **tokens)
diff --git a/src/print_ast.c b/src/print_ast.c
index 94f813f..a511246 100644
--- a/src/print_ast.c
+++ b/src/print_ast.c
@@ -6,7 +6,7 @@
 /*   By: dkaiser <dkaiser@student.42heilbronn.de    +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2024/07/22 15:16:53 by dkaiser           #+#    #+#             */
-/*   Updated: 2024/08/02 13:18:48 by dkaiser          ###   ########.fr       */
+/*   Updated: 2024/08/11 12:26:00 by dkaiser          ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -49,14 +49,6 @@ static void	print_cmd_node(t_node *ast, int indent)
 		printf(" '%s'", ast->content.cmd.args[i]);
 		i++;
 	}
-	i = 0;
-	printf("\n%*sASSIGNS:", indent + 2, "");
-	while (ast->content.cmd.assigns[i] != NULL)
-	{
-		printf(" %s=%s", ast->content.cmd.assigns[i]->var,
-			ast->content.cmd.assigns[i]->value);
-		i++;
-	}
 	printf("\n%*sREDIRS:", indent + 2, "");
 	printf("\n%*sIN: %d %s", indent + 4, "", ast->content.cmd.redirs[0].type,
 		ast->content.cmd.redirs[0].specifier);
-- 
cgit v1.2.3