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

Fix compiler warning #389

Closed
wants to merge 2 commits into from
Closed

Fix compiler warning #389

wants to merge 2 commits into from

Conversation

probonopd
Copy link
Owner

@probonopd probonopd commented Dec 9, 2022

@probonopd
Copy link
Owner Author

Still getting:

In file included from minidexed.cpp:20:
minidexed.h: In constructor 'CMiniDexed::CMiniDexed(CConfig*, CInterruptSystem*, CGPIOManager*, CI2CMaster*, FATFS*)':
minidexed.h:300:7: warning: 'CMiniDexed::m_bSavePerformance' will be initialized after [-Wreorder]
  300 |  bool m_bSavePerformance;
      |       ^~~~~~~~~~~~~~~~~~
minidexed.h:299:7: warning:   'bool CMiniDexed::m_bSavePerformanceNewFile' [-Wreorder]
  299 |  bool m_bSavePerformanceNewFile;
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~
minidexed.cpp:33:1: warning:   when initialized here [-Wreorder]
   33 | CMiniDexed::CMiniDexed (CConfig *pConfig, CInterruptSystem *pInterrupt,
      | ^~~~~~~~~~
In file included from minidexed.cpp:20:
minidexed.h:299:7: warning: 'CMiniDexed::m_bSavePerformanceNewFile' will be initialized after [-Wreorder]
  299 |  bool m_bSavePerformanceNewFile;
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~
minidexed.h:261:7: warning:   'bool CMiniDexed::m_bSetNewPerformance' [-Wreorder]
  261 |  bool m_bSetNewPerformance;
      |       ^~~~~~~~~~~~~~~~~~~~
minidexed.cpp:33:1: warning:   when initialized here [-Wreorder]
   33 | CMiniDexed::CMiniDexed (CConfig *pConfig, CInterruptSystem *pInterrupt,
      | ^~~~~~~~~~

@probonopd
Copy link
Owner Author

I don't know how to fix this.

@probonopd
Copy link
Owner Author

Closig in favor of #390

@probonopd probonopd closed this Dec 10, 2022
@probonopd probonopd deleted the fix-387 branch December 10, 2022 14:56
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.

1 participant