Timeline for pset5-speller: dependency issue
Current License: CC BY-SA 3.0
10 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jan 24, 2017 at 9:10 | comment | added | Blauelf |
Removed quotes from "dictionary"? Also, the current state of the code includes an fopen with "w+". There should be only one fopen, with mode "r". As @DinoCoderSaurus pointed out, never write to the input dictionary file. And there are funny infinite loops. !&feof is probably false, so those loops won't run, therefore infinity does not apply. I suggest doing a dry run on paper first.
|
|
| Jan 24, 2017 at 3:04 | history | edited | DinoCoderSaurus | CC BY-SA 3.0 |
more
|
| Jan 24, 2017 at 3:00 | comment | added | DinoCoderSaurus | clang is not claiming there are "unused arguments" in the code. The -Qunused-arguments switch suppresses compiler warnings about unused arguments in the clang command line, not arguments in the code. Answer emended. | |
| Jan 24, 2017 at 0:19 | comment | added | doublestanding | Can unused arguments make a function to fail, because, it seems Clang is claiming that there are unused arguments in the code. I thought unused arguments are redundant in a code. | |
| Jan 24, 2017 at 0:16 | comment | added | doublestanding | clang -ggdb3 -O0 -Qunused-arguments -std=c11 -Wall -Werror -c -o speller.o speller.c clang -ggdb3 -O0 -Qunused-arguments -std=c11 -Wall -Werror -c -o dictionary.o dictionary.c clang -ggdb3 -O0 -Qunused-arguments -std=c11 -Wall -Werror -o speller speller.o dictionary.o ~ Usage: speller [dictionary] text: Can this be of any diagnostic value? | |
| Jan 23, 2017 at 23:43 | history | edited | DinoCoderSaurus | CC BY-SA 3.0 |
+comments
|
| Jan 23, 2017 at 23:41 | comment | added | DinoCoderSaurus | Another quick scan: when and where does the function return true? You should be able to use debug50. set a breakpoint at the beginning of load in dictionary.c and walk it through. (small dictionary might be easier to start with) | |
| Jan 23, 2017 at 23:36 | comment | added | doublestanding | Thanks, @DinoCoderSaurus, I have implemented those changes, but nothing has changed, except loss of memory at line 81(which initially was not there], upon the fact that I freed &root and &branches[index]. About going to chck50, I did it out of exasperation, to know if I can get some diagnostic information from therein! | |
| Jan 23, 2017 at 22:34 | history | edited | DinoCoderSaurus | CC BY-SA 3.0 |
+warning
|
| Jan 23, 2017 at 22:01 | history | answered | DinoCoderSaurus | CC BY-SA 3.0 |