Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • A assignment-image-rotation
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 23
    • Merge requests 23
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • c-language
  • assignment-image-rotation
  • Issues
  • #1

Closed
Open
Created Jan 25, 2021 by Dmitry Kupershtein@kupp0 of 5 tasks completed0/5 tasks

Follow-up from "Master"

The following discussions from !49 (closed) should be addressed:

  • @sayon started a discussion:

    Пользователю трансформаций не хочется вообще знать про заголовки bmp файлов, ему это не нужно. Он хочет просто из struct image сделать bmp файл. Поэтому заголовок надо скрыть от пользователя. Как следствие функции типа read_header, если и есть, не должны выставляться через заголовочный файл для других частей программы. Определяете в .c файле и помечаете static.

  • @sayon started a discussion:

    • Типы у height и i разные.
  • @sayon started a discussion:

    • bmp_header маленький, всего-то 60 байт. Можно не создавать его в куче, а выделить в стеке.
  • @sayon started a discussion:

    static char const* const :)

  • @sayon started a discussion:

    проверку на корректность заголовка разумно вынести в отдельную функцию.

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking