Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#722 closed ожидается проверка (задача сдана)

HW_02

Reported by: bobrov.sergej Owned by: Святослав Власов
Component: HW #1 (BMP) Version: 1.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed

Где Makefile? У себя вы можете собирать какой угодно системой сборки, но тестирующая система ожидает Makefile.

Твой CMakeLists.txt тоже не собирает ничего, похоже ты забыл добавить HuffmanTree.h в свой репозиторий.

[558][svloyso:out]$ cmake ..
-- The C compiler identification is GNU 10.2.0
-- The CXX compiler identification is GNU 10.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
CMake Error at CMakeLists.txt:11 (add_executable):
  Cannot find source file:

    include/HuffmanTree.h

  Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .h .hh .h++
  .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .ispc


CMake Error at CMakeLists.txt:11 (add_executable):
  No SOURCES given to target: hw_03


CMake Generate step failed.  Build files cannot be regenerated correctly.

Поэтому корректность проверить не могу. Можешь дослать недостающий файл и переоткрыть тикет, я допроверю корректность.

Стиль:

  1. Про константу CHAR_BIT ты знаешь, но почему-то её не используешь там, где она нужна.
  2. frequencies{std::vector<uint64_t>(UCHAR_MAX + 1): std::array же! Вектор у тебя статический.
  3. Аналогично std::vector<Bits::Bits> codes(256);

0/5/9

comment:2 Changed 3 years ago by Святослав Власов

Допроверил корректность.
Валгринд возмущается на ошибки при компрессии-декомпрессии файла содержащего последовательно все файлы от 0x00 до 0xff и некоторых других.

10/5/9

Last edited 3 years ago by Святослав Власов (previous) (diff)
Note: See TracTickets for help on using tickets.