PDL git repo | Cypress Semiconductor

PDL git repo

Summary: 2 Replies, Latest post by Lorenzo Cappelletti on 23 Nov 2016 02:17 AM PST
Verified Answers: 0
Last post
Log in to post new comments.
Lorenzo Cappelletti's picture
User
5 posts

Hi there,

I'd like to contribute back to PDL with a couple of simple patches for fixing warnings when compiling with GCC. Is there any git repository holding PDL source code?

On the contrary, where can I send my patch to?

Thanks,

Lorenzo

 

PS: at the moment, I'm using

arm-none-eabi-g++.exe (GNU Tools for ARM Embedded Processors) 5.4.1 20160609 (release) [ARM/embedded-5-branch revision 237715]

user_1377889's picture
User
9583 posts

Welcome in the forum, Lorenzo.

There is a thread for Software -> Community components which would be a good place for well-documented patches.

I personally have a tendency to change my programs, so that all warnings are suppressed. Sometimes the warnings hint me that I have done something wrong which I need to correct. So what are your patches actually performing?

 

Bob

Lorenzo Cappelletti's picture
User
5 posts

I totally agree with you: I do dislike suppressing warnings. Nevertheless, some of them are very annoying.

The warnings I'm suppressing are:

  • missing quotes around #error message string
  • -Wunused-parameter
  • -Wunused-but-set-variable
  • -Wparentheses (because of v = v & ~x | y which becomes v = (v & ~x) | y)
  • -Wconversion
  • -Wpadded
Log in to post new comments.