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

ported lab03/samples_old -> lab02/image_processing #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

8migrannik
Copy link
Contributor

No description provided.

@alexey-malov
Copy link
Owner

------ Build started: Project: sample03_blur_filter_wtl_optimized_openmp, Configuration: Debug Win32 ------
cl : command line  warning D9035: option 'Gm' has been deprecated and will be removed in a future release
...
AboutDlg.obj : warning LNK4075: ignoring '/EDITANDCONTINUE' due to '/SAFESEH' specification

Надо исправить warning-и. Во всех конфигурациях и платформах проекты должны компилироваться без предупреждений.

@alexey-malov
Copy link
Owner

Примеры sample02_blur_filter_wtl_optimized и sample03_blur_filter_wtl_optimized_openmp имеют в PR одинаковый исходный код во всех файлах, тогда как в оригинальном репозитории они отличались в MatrixFilter.cpp

@alexey-malov
Copy link
Owner

Примеры относятся, скорее ко 2 лабораторной работе, так как не используют OpenGL, а иллюстрируют идеи цифровой обработки изображений на CPU. Раньше для этого у меня была отдельная лабораторная работа №3.

Нужно перенести PR в лабораторную работу в отдельный каталог с примерами labs/02/image_processing (там солюшен должен называться image_processing.sln)

@alexey-malov
Copy link
Owner

При запуске скомпилированного EXE не через IDE, а из выходного каталога, картинки не отображаются, так как файлы ресурсов не были скопированы в выходной каталог. Посмотрите в README.md в корне репозитория информацию о том, как правильно копировать файлы ресурсов в выходной каталог


LRESULT OnPaint(UINT /*uMsg*/, WPARAM /*wParam*/, LPARAM /*lParam*/, BOOL& /*bHandled*/);
private:
// �������� �����������
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот и другие файлы .h/.cpp следует сохранить в кодировке UTF-8 without BOM

Comment on lines 45 to 47
std::bind1st(
std::multiplies<float>(),
scale) // bind1st ������� ������� ������� ��
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот код перестанет компилироваться после перехода на C++17, где bind1st и bind2nd выпилили. Нужно просто заменить на такую лямбду:

[scale](float value) { return scake * value; }

В остальных проектах сделать так же.

LRESULT OnPaint(UINT /*uMsg*/, WPARAM /*wParam*/, LPARAM /*lParam*/, BOOL& /*bHandled*/);
private:
// �������� �����������
std::auto_ptr<Gdiplus::Bitmap> m_pSrcPicture;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auto_ptr был удалён в C++17. При переходе на новый стандарт C++ код не будет компилироваться. Нужно во всех местах заменить auto_ptr на unique_ptr

@8migrannik 8migrannik requested a review from alexey-malov June 15, 2023 18:58
@8migrannik 8migrannik changed the title ported lab03/samples_old -> lab05/samples_win_32 ported lab03/samples_old -> lab02/image_processing_32 Jun 15, 2023
@8migrannik 8migrannik changed the title ported lab03/samples_old -> lab02/image_processing_32 ported lab03/samples_old -> lab02/image_processing Jun 15, 2023
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.

2 participants