![]() |
Please don't reformat WPILib code during build season
I'm not sure who is working on WPILib, but it would be very nice if the mid-season releases didn't change source code formatting. I've been trying to keep up with bugs and fixes, but the cosmetic changes make the fixes harder to find.
Thank you for the source code releases by the way. It's incredibly valuable to be able to see how WPILib works rather than it being a black box. |
Re: Please don't reformat WPILib code during build season
Quote:
|
Re: Please don't reformat WPILib code during build season
Quote:
On a more personal level, I'd like to be a part of the community. This is an incredible Open Source project with active and inspired development. In order for me to be effective, it's my responsibility to understand the code. I'm asking for a professional courtesy to help make that job easier. You can see the changes in the latest WPILib here: https://github.com/FRC3322/WPILib/co...688e34f4b13ee1 The green lines are additions; the red lines deletions. |
Re: Please don't reformat WPILib code during build season
After looking at your diff I see that you are just making the point that if (for example) a curly brace is moved from right after the method arguments to the next line it shows up as a difference which makes it more difficult to determine what functionaly changed in the code.
Not sure if you are aware but you can track (and submit) bugs and their fixes on the FIRST Forge site. http://firstforge.wpi.edu/sf/tracker....wpilib_c_bugs |
| All times are GMT -5. The time now is 12:55. |
Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.
Copyright © Chief Delphi