{"push_id":757692302,"size":22,"distinct_size":0,"ref":"refs/heads/ModifiedControlVariableFixes","head":"ff860c222cfc2c6a3c4cc2eeeae8af3c8eb0407c","before":"eeaa0c4399e433e8b0feba33168c46e37c7c6a27","commits":[{"sha":"82d8e91024a57a96fbb6410dacb29d6a3f582903","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Remove unnecessary qualifiers. #1555\n\nFixes `UnnecessarilyQualifiedStaticUsage` inspection violations.\n\nDescription:\n>Reports calls to static methods or accesses of static fields on the current class which are qualified with the class name. Such qualification is unnecessary, and may be safely removed.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/82d8e91024a57a96fbb6410dacb29d6a3f582903"},{"sha":"32bd618b193dc04935b9c3c11b5bc4364cca8c89","author":{"email":"b948357777583d3727f4d6aef60d25bcea77ca22@gmail.com","name":"Baratali Izmailov"},"message":"Issue #1243: Generic types should be parameterized","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/32bd618b193dc04935b9c3c11b5bc4364cca8c89"},{"sha":"e13ff7caab177320a0ce2c58d7e6ec3263aaff32","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Fix headers for classes in 'gui' package. #945","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/e13ff7caab177320a0ce2c58d7e6ec3263aaff32"},{"sha":"0da6b95ae7af2def88f785b72f886dd418a89d43","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Prevent classes to escape their scope. #1555\n\nFixes `ClassEscapesItsScope` inspection violation.\n\nDescription:\n>Reports any references to classes which allow the class name to be used outside the class's stated scope. For instance, this inspection would report a public method which returns a private inner class, or a protected field whose type is a package-visible class. While legal Java, such references can be very confusing, and make reuse difficult.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/0da6b95ae7af2def88f785b72f886dd418a89d43"},{"sha":"ba9639e5decde52e38fbc1a048be7501dc701ac3","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Move constants to right side in comparisons. #1555\n\nFixes ` ConstantOnLHSOfComparison` inspection violations.\n\nDescription:\n>Reports on comparison operations with constant values on their left-hand side. Some coding conventions specify that constants should be on the right-hand side of comparisons.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/ba9639e5decde52e38fbc1a048be7501dc701ac3"},{"sha":"c4928f486c67afa62e7e8fbd5d8f731be7186985","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Decrease scope of variables. #1555\n\nFixes `TooBroadScope` inspection violations.\n\nDescription:\n>Reports any variable declarations of which the scope can be narrowed. Especially useful for \"Pascal style\" declarations at the start of a method, but variables with too broad a scope are also often left over after refactorings.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/c4928f486c67afa62e7e8fbd5d8f731be7186985"},{"sha":"ea06b2fb96f2120ceee04a998021c1dc752e25d3","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Decrease visibility of inner classes. #1555\n\nFixes `PackageVisibleInnerClass` inspection violation.\n\nDescription:\n>Reports package-local inner classes.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/ea06b2fb96f2120ceee04a998021c1dc752e25d3"},{"sha":"fca2fd4d07ad057664ec43b83a3835693a1b1c88","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Extract nested class with other class nested to separate file. #1555\n\nFixes `ClassNestingDepth` inspection violation.\n\nDescription:\n>Reports inner classes too deeply nested. Nesting inner classes inside inner classes is almost certain to be confusing, and is a good sign that refactoring may be necessary.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/fca2fd4d07ad057664ec43b83a3835693a1b1c88"},{"sha":"3bd699bd593414d63c7139aa410f13ec8341a027","author":{"email":"ac9a121066ac44464c4c9521e3e4158acf341152@gmail.com","name":"Ruslan Diachenko"},"message":"Issue #1566: HiddenField violations fixed","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/3bd699bd593414d63c7139aa410f13ec8341a027"},{"sha":"289ef67e9e9ee424723b8aa22ae7a6d31f126d23","author":{"email":"90395d3a7bbf3ec501c1608e71fc602eb7c65c8d@gmail.com","name":"Andrei Selkin"},"message":"Marked 'maxLineLength' as '@Deprecated' at LeftCurlyCheck, issue #965","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/289ef67e9e9ee424723b8aa22ae7a6d31f126d23"},{"sha":"8a6f12d9e9f4f5e9f9b9a04dfa5be8d42ad747c7","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Decrease visibility of constructor in abstract class. #1555\n\nFixes `NonProtectedConstructorInAbstractClass` inspection violation.\n\nDescription:\n>Reports constructors in abstract classes that are not declared protected, package-protected or private.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/8a6f12d9e9f4f5e9f9b9a04dfa5be8d42ad747c7"},{"sha":"331992aacd23576c72862ea75d83f15e09af958c","author":{"email":"7feb7b10c89ca00f910c413dba7eeb8da603b434@gmail.com","name":"Ilja Dubinin"},"message":"Coverage has been increased to 100% in CommentSuppressor. Issue #1010","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/331992aacd23576c72862ea75d83f15e09af958c"},{"sha":"9a582720939c1b4098430d1ab439b9ce045eaf06","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Remove redundant throws declarations. #1555\n\nFixes `RedundantThrowsDeclaration` inspection violations.\n\nDescription:\n>This inspection reports exceptions that are declared in a method's signature but never thrown by the method itself.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/9a582720939c1b4098430d1ab439b9ce045eaf06"},{"sha":"26ffb0af87d8718dd58344d6a95df477fa66f380","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Change name of private method conflicting with superclass method. #1555\n\nFixes `MethodOverridesPrivateMethod` inspection violations.\n\nDescription:\n>Reports instance methods having the same name as a private method of a superclass. Such methods may result in confusing semantics, particularly if the private method is ever made publicly visible.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/26ffb0af87d8718dd58344d6a95df477fa66f380"},{"sha":"677a74deee0451f6936aace79a58890cf7d27837","author":{"email":"ac9a121066ac44464c4c9521e3e4158acf341152@gmail.com","name":"Ruslan Diachenko"},"message":"Issue #1566: ImportOrder violations fixed","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/677a74deee0451f6936aace79a58890cf7d27837"},{"sha":"29f945e338d6cd9d6382312b54965be2976a5d2a","author":{"email":"ac9a121066ac44464c4c9521e3e4158acf341152@gmail.com","name":"Ruslan Diachenko"},"message":"Issue #1566: Constructor definition in wrong order violations fixed","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/29f945e338d6cd9d6382312b54965be2976a5d2a"},{"sha":"7b34e9d7cd424732e7a9035f8ba795369157d6ab","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Do not allocate arrays of zero length. #1555\n\nFixes `ZeroLengthArrayInitialization` inspection violations.\n\nDescription:\n>Reports on allocations of arrays with known lengths of zero. Since array lengths in Java are non-modifiable, it is almost always possible to share zero-length arrays, rather than repeatedly allocating new zero-length arrays. Such sharing may provide useful optimizations in program runtime or footprint. Note that this inspection does not report zero-length arrays allocated as static final fields, as it is assumed that those arrays are being used to implement array sharing.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/7b34e9d7cd424732e7a9035f8ba795369157d6ab"},{"sha":"e9ade4ddad669b97ec01ab70d59bd6c3635616ed","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Remove redundant unit in zero value form CSS. #1555\n\nFixes `CssRedundantUnit` inspection violations.\n\nDescription:\n>This inspection highlights zero values with specified unit of measurement.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/e9ade4ddad669b97ec01ab70d59bd6c3635616ed"},{"sha":"6a0bad784f056988b3b84b75baa397ac78102a24","author":{"email":"4c0cab09f7c6d64360f84901b2eed93d1d93f05b@gmail.com","name":"Michal Kordas"},"message":"Remove unnecessary this. #1555\n\nFixes `UnnecessaryThis` inspection violations after recent commits.\n\nDescription:\n>Reports on any unnecessary uses of this in the code. Using this to disambiguate a code reference may easily become unnecessary via automatic refactorings, and is discouraged by many coding styles.","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/6a0bad784f056988b3b84b75baa397ac78102a24"},{"sha":"7770f2a2c25c41bf024355f19caac358419110a6","author":{"email":"7feb7b10c89ca00f910c413dba7eeb8da603b434@gmail.com","name":"Ilja Dubinin"},"message":"Loops have to have only one breal/continue. Issue #46","distinct":false,"url":"https://api.github.com/repos/rdiachenko/checkstyle/commits/7770f2a2c25c41bf024355f19caac358419110a6"}]}