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

Modification of source code to comply with C99 and C++11 standards #8

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

Conversation

homewsn
Copy link

@homewsn homewsn commented Feb 19, 2024

To prepare for using another toolchain, I tried to bring your source code as much as possible to the C99 and C++11 standards. Perhaps this will be interesting for you.

…tion 'strchr' (a value of type "int" cannot be assigned to an entity of type "char const *")
…(a value of type "Widget *" cannot be used to initialize an entity of type "MenuEntry *")
…ned int*}' to 'long int*' (no instance of constructor "MenuValue<T>::MenuValue [with T=int32_t]" matches the argument list)
@jankae
Copy link
Owner

jankae commented Feb 20, 2024

Hi, thanks for all the work. I am a bit unsure what I am going to do with this PR because:

  • I don't want to merge something I haven't at least compiled and tested myself
  • My LCR meter is working fine as it is
  • I am very much buried in work right now and don't really have time to check on some project that I haven't actively developed in years (although I still use it from time to time)

So I think I will leave this open for a couple of weeks, wait for me to find time to properly check it and merge it then. Sorry, the LCR is a very low priority item on my list right now.
But I also don't want to throw away your improvements, so should I forget about this please remind me in a few weeks.

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