Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixing intf6_.mod for NEURON 9.0.0, 8.2.0, 8.1.0 #1

Merged
merged 17 commits into from
May 27, 2022
Merged

Fixing intf6_.mod for NEURON 9.0.0, 8.2.0, 8.1.0 #1

merged 17 commits into from
May 27, 2022

Conversation

iomaganaris
Copy link
Contributor

@iomaganaris iomaganaris commented May 16, 2022

Only warning is:

intf6_.c: In function ‘qstats_INTF6’:
intf6_.c:2994:27: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘unsigned int’ [-Wformat=]
 2994 |     printf("SPIKES: %d (%ld:%ld)\n",IDP->spkcnt,spikes[lct],blockcnt[lct]);
      |                         ~~^                     ~~~~~~~~~~~
      |                           |                           |
      |                           long int                    unsigned int
      |                         %d
intf6_.c:2994:31: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘unsigned int’ [-Wformat=]
 2994 |     printf("SPIKES: %d (%ld:%ld)\n",IDP->spkcnt,spikes[lct],blockcnt[lct]);
      |                             ~~^                             ~~~~~~~~~~~~~
      |                               |                                     |
      |                               long int                              unsigned int
      |                             %d
intf6_.c:2997:34: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘unsigned int’ [-Wformat=]
 2997 |       fprintf(tfo,"SPIKES: %d (%ld:%ld);",IDP->spkcnt,spikes[lct],blockcnt[lct]);
      |                                ~~^                    ~~~~~~~~~~~
      |                                  |                          |
      |                                  long int                   unsigned int
      |                                %d
intf6_.c:2997:38: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘unsigned int’ [-Wformat=]
 2997 |       fprintf(tfo,"SPIKES: %d (%ld:%ld);",IDP->spkcnt,spikes[lct],blockcnt[lct]);
      |                                    ~~^                            ~~~~~~~~~~~~~
      |                                      |                                    |
      |                                      long int                             unsigned int
      |                                    %d

Maybe this shouldn't be changed to avoid messing up the output?

@ramcdougal
Copy link

Can you also add a note to the readme indicating that the file has changed; see e.g. https://senselab.med.yale.edu/ModelDB/showmodel?model=2488&file=/cells/README.txt#tabs-2

@iomaganaris iomaganaris marked this pull request as ready for review May 17, 2022 16:06
@ramcdougal ramcdougal merged commit fd6ad41 into ModelDBRepository:master May 27, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants