Workbranch#1
Conversation
| language: c | ||
|
|
||
| os: | ||
| - osx |
There was a problem hiding this comment.
все изменения должны были быть отображены в Pull Request
сейчас это не так
| - gcc -std=c11 --coverage -c *.c | ||
| #- gcc -o Homework2 *.o -fprofile-arcs -ftest-coverage -g -L. -ldyn -lstat -lpthread -Wl,-rpath,. | ||
| #- ./Homework2 | ||
| - lcov -t "coverage" -o coverage.info -c -d . |
There was a problem hiding this comment.
в Merge Request не приведены сведения по покрытию кода тестами
|
|
||
| add_executable(Homework2 main.c dynamic_library/dyn_lib.c dynamic_library/dyn_lib.h static_library/stat_lib.c static_library/stat_lib.h) No newline at end of file | ||
| find_package (Threads) | ||
| add_executable(Homework2 main.c libdyn.c libdyn.h libstat.c libstat.h) |
There was a problem hiding this comment.
интерфейс обеих библиотек должен быть одинаковым
поскольку подключается обе библиотеки, то значит, что ODR правило не нарушается => интерфейс различный
| @@ -193,3 +193,41 @@ Linking C executable cmTC_d98ef | |||
| Feature record: C_FEATURE:0c_restrict | |||
There was a problem hiding this comment.
папки cmake-build-debug быть не должно
| #include <stdlib.h> | ||
| #include <pthread.h> | ||
| #include <zconf.h> | ||
| #include "libdyn.h"//NOLINT |
There was a problem hiding this comment.
почему тут используется NOLINT?
| @@ -1,6 +1,52 @@ | |||
| /* | |||
There was a problem hiding this comment.
не приведено условие задачи
| int res_mt = mt_run(sequence, size, log_path); | ||
| int res_ot = run(sequence, size, log_path); | ||
| printf("%s %d %s %d", "test mt:", res_mt, " ot:", res_ot); | ||
| assert(res_mt == res_ot && res_mt == answer); |
There was a problem hiding this comment.
assert работает только в DEBUG-режиме
| * "Copyright 2019 Sergei Petrenko" | ||
| */ | ||
|
|
||
| #include "stat_lib.h" |
There was a problem hiding this comment.
что этот файл здесь делает?
| sequence[i++] = c; | ||
| if (i == seq_size) { | ||
| seq_size = i + len_max; | ||
| sequence = realloc(sequence, seq_size); |
There was a problem hiding this comment.
если realloc упадёт - будет утечка памяти
| char *sequence = malloc(len_max); | ||
| seq_size = len_max; | ||
| printf("\nEnter a string:"); | ||
| char c = EOF; |
There was a problem hiding this comment.
EOF умещается в char?
getchar возвращает char?
2 participants
Не смог разобраться с ошибкой. При компиляции библиотеки выдает sigfault.