Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2 minishell windows bugs #2

Merged
merged 6 commits into from
Mar 26, 2017
Merged

2 minishell windows bugs #2

merged 6 commits into from
Mar 26, 2017

Conversation

SebiSebi
Copy link
Contributor

In legatura cu "Remove unnecessary headers in main.c" daca incerci sa compilezi repo-ul fara sa faci nimic
(git clone la so-assignment) primesti niste erori de compilare ciudate prin utils-win.c. Am cautat putin pe net si problema provine de la anumite dependente circulare, cumva, desi nu mi-a fost clar de ce garzile din .h-uri nu isi fac treaba cum trebuie. Mi-a luat ceva pana sa imi dau seama ca vine de la main.c pentru ca nu zice nimic de asta in eroare.

lukablurr
lukablurr previously approved these changes Mar 26, 2017
@lukablurr lukablurr dismissed their stale review March 26, 2017 12:10

need discussion on 2 changes

@lukablurr
Copy link

Comentarii per commit:

  1. Ok
  2. De ce ai nevoie să faci fflush la stderr? AFAIK, stderr nu este buffered. Dacă nu e nevoie atunci facem revert la commit.
  3. Deși modificarea ta rezolvă erorile, nu e chiar modificarea optimă; ai putea avea nevoie de utils în main.c. Patch-ul corect pentru acest commit ar fi cel de mai jos. Te rog, fă revert la commit și aplică modificările de mai jos.
diff --git a/2-minishell/skel-windows/utils.h b/2-minishell/skel-windows/utils.h
index 13284a8..0bfb1fe 100644
--- a/2-minishell/skel-windows/utils.h
+++ b/2-minishell/skel-windows/utils.h
@@ -6,8 +6,10 @@
 #ifndef _UTILS_H
 #define _UTILS_H
 
+#include <windows.h>
 #include "parser.h"
 
+VOID PrintLastError(const PCHAR message);
 
 /* useful macro for handling error codes */
 #define DIE(assertion, call_description) \

@SebiSebi
Copy link
Contributor Author

  1. Da, asa este. Eu am reusit sa produc un comportament de genul: https://snag.gy/QyBL5V.jpg si initial am crezut ca afisarea erori dupa exit provine de acolo, dar nu este asa (parser-ul intoarce NULL pentru un input eronat, dar eroarea se afiseaza la urmatorul apel).
  2. Ok.

@lukablurr lukablurr merged commit 107019c into systems-cs-pub-ro:master Mar 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants