<nb>
 COMMENT
 COMMENT
 COMMENT
 - package NAMESPACE ;
 + package NAMESPACE ;

 import static NAMESPACE ;

 <nb> import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

moved LoadTest to com mysema query collections


<nb> final class ConfigSubstitution extends AbstractConfigValue implements
 @ override
 public ConfigValueType valueType ( ) {
 throw new ConfigException . NotResolved (
 - STRING0
 + STRING1
 + this ) ;
 }

 @ override
 public Object unwrapped ( ) {
 throw new ConfigException . NotResolved (
 - STRING2 + this ) ;
 + STRING3
 + + this ) ;
 }

 @ override

make unresolved subst exception suggest calling resolve


<nb> public class OConsoleDatabaseApp extends OrientConsole implements OCommandOutput
 resultSet . add ( row ) ;

 row . field ( STRING0 , p . getName ( ) ) ;
 - row . field ( STRING1 , p . getType ( ) ) ;
 + row . field ( STRING1 , ( Object ) p . getType ( ) ) ;
 row . field ( STRING2 , p . getLinkedClass ( ) != null ? p . getLinkedClass ( ) : p . getLinkedType ( ) ) ;
 row . field ( STRING3 , p . isMandatory ( ) ) ;
 row . field ( STRING4 , p . isReadonly ( ) ) ;


<nb> public final class SessionFactoryImpl
 this . currentSessionContext = null ;
 this . sqlFunctionRegistry = null ;
 this . transactionEnvironment = null ;
 + this . jdbcServices = null ;
 + this . dialect = null ;

 this . sessionFactoryOptions = sessionFactoryOptions ;

Fixed a problem with a conflict with Gale s change to this class


<nb> public class D2RVisitorCallbackImpl implements Callback < DocPartRowImpl > , Collect

 public void visit ( KVArray array , AttributeReference attributeReference , TableRef tableRef ,
 DocPartRowImpl parentRow ) {
 - if ( array . isEmpty ( ) ) {
 - return ;
 - }
 -
 attributeReference = attributeReference . append ( new ArrayKey ( NUMBER0 ) ) ;
 tableRef = attributeReferenceTranslator . toTableRef ( attributeReference ) ;
 DocPartDataImpl docPartData = getDocPartData ( attributeReference , tableRef ) ;

Fixed table not created for an empty array


<nb> public abstract class AbstractBlackBoxTestCase extends TestCase {

 int totalTests = imageFiles . length * testCount * NUMBER0 ;
 System . out . println ( STRING0 + totalFound + STRING1 + totalTests +
 - STRING2 + ( totalFound * NUMBER1 / totalTests ) + STRING3 ) ;
 + STRING2 + ( totalFound * NUMBER1 / totalTests ) + STRING4 + totalMustPass + STRING5 ) ;
 if ( totalFound > totalMustPass ) {
 System . out . println ( STRING6 + ( totalFound - totalMustPass ) + STRING7 ) ;
 + } else if ( totalFound < totalMustPass ) {
 + System . out . println ( STRING8 + ( totalMustPass - totalFound ) + STRING7 ) ;
 }

 SINGLE

Added slightly better blackbox unit test stats


<nb> package NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;

 COMMENT
 COMMENT
 COMMENT
 public class PigTestUtil {

 - private static final Logger LOG = LoggerFactory . getLogger ( PigTestUtil . class ) ;
 -
 -
 COMMENT
 COMMENT
 COMMENT


<nb> public class BenchmarkRunner
 Thrift . register ( groups ) ;
 ActiveMQProtobuf . register ( groups ) ;
 Kryo . register ( groups ) ;
 - CksBinary . register ( groups ) ;
 AvroSpecific . register ( groups ) ;
 AvroGeneric . register ( groups ) ;
 Hessian . register ( groups ) ;
 <nb> public class BenchmarkRunner
 JsonJacksonDatabind . register ( groups ) ;
 JsonTwoLattes . register ( groups ) ;
 ProtostuffJson . register ( groups ) ;
 - CksText . register ( groups ) ;
 ProtobufJson . register ( groups ) ;
 Gson . register ( groups ) ;


<nb> public class RemoteDriver {
 if ( sparkCounters != null ) {
 counters = sparkCounters . snapshot ( ) ;
 }
 + SINGLE
 + SINGLE
 + for ( JavaFutureAction < ? > future : jobs ) {
 + future . get ( ) ;
 + }
 protocol . jobFinished ( req . id , result , null , counters ) ;
 } catch ( Throwable t ) {
 SINGLE

Failed job may not throw exceptions Spark Branch Rui via Xuefu


<nb> public class Fragment implements ComponentCallbacks2 , OnCreateContextMenuListene
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public class Fragment implements ComponentCallbacks2 , OnCreateContextMenuListene
 }

 COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public void onDetach ( ) {
 mCalled = true ;

Clarified usage of onDetach and onCreate when setRetainInstance is set to true


<nb> class MockTaskManager implements TaskOperationProtocol {
 public void recoverAll ( ChannelID sourceChannelID ) {
 }

 + @ override
 + public void restart ( ExecutionVertexID executionVertexID ) {
 + }
 +
 }


<nb> public class PsiNamesElementSignatureProvider extends AbstractElementSignaturePr
 }
 return null ;
 }
 -
 - int index = Integer . parseInt ( tokenizer . nextToken ( ) ) ;
 - return restoreElementInternal ( parent , type , index , PsiNamedElement . class ) ;
 +
 + try {
 + int index = Integer . parseInt ( tokenizer . nextToken ( ) ) ;
 + return restoreElementInternal ( parent , type , index , PsiNamedElement . class ) ;
 + }
 + catch ( NumberFormatException e ) {
 + return null ;
 + }
 }

 @ override

Provide generic mechanism that allows folding state to survive IDE restart


<nb> public class DynamicRecordFormat extends BaseOneByteHeaderRecordFormat < DynamicRe

 readData ( record , cursor ) ;
 }
 + else
 + {
 + record . setInUse ( inUse ) ;
 + }
 }

 public static String payloadTooBigErrorMessage ( DynamicRecord record , int recordSize , int nrOfBytes )
 <nb> public abstract class RecordFormatTest
 private static final int PAGE_SIZE = ( int ) kibiBytes ( NUMBER0 ) ;

 SINGLE
 - private static final long TEST_ITERATIONS = NUMBER1 ;
 - private static final long TEST_TIME = NUMBER2 ;
 + private static final long TEST_ITERATIONS = NUMBER3 ;
 + private static final long TEST_TIME = NUMBER4 ;
 private static final long PRINT_RESULTS_THRESHOLD = SECONDS . toMillis ( NUMBER0 ) ;
 private static final int DATA_SIZE = NUMBER5 ;
 protected static final long NULL = Record . NULL_REFERENCE . intValue ( ) ;

Fix a bug in decoding of dynamic records in the standard format


<nb> public class PrivacyProvider extends ContentProvider {

 SharedPreferences prefs = context . getSharedPreferences ( PREF_SETTINGS + STRING0 + uid ,
 Context . MODE_WORLD_READABLE ) ;
 - for ( String settingKey : prefs . getAll ( ) . keySet ( ) )
 + for ( String name : prefs . getAll ( ) . keySet ( ) )
 try {
 - String name = getSettingName ( settingKey ) ;
 - String value = prefs . getString ( settingKey , STRING1 ) ;
 + String value = prefs . getString ( name , STRING1 ) ;
 listWork . add ( new ParcelableSetting ( uid , name , value ) ) ;
 } catch ( Throwable ex ) {
 SINGLE


<nb> public class SvnFileSystemListener implements LocalFileOperationsHandler , Comman
 if ( rootManager != null ) {
 Module module = rootManager . getFileIndex ( ) . getModuleForFile ( file ) ;
 if ( module != null ) {
 - AbstractVcs vcs = ModuleLevelVcsManager . getInstance ( module ) . getActiveVcs ( ) ;
 + AbstractVcs vcs = ProjectLevelVcsManager . getInstance ( project ) . getVcsFor ( file ) ;
 if ( vcs instanceof SvnVcs ) {
 return ( SvnVcs ) vcs ;
 }


<nb> public class UsageDescriptor implements Comparable < UsageDescriptor > {
 public int compareTo ( UsageDescriptor ud ) {
 return getKey ( ) . compareTo ( ud . myKey ) ;
 }
 +
 + @ override
 + public String toString ( ) {
 + return myKey + STRING0 + myValue ;
 + }
 }


<nb> public class HtmlUtil {
 SINGLE
 STRING0 ,
 SINGLE
 - STRING1 , STRING2 , STRING3 , STRING4 , STRING5 , STRING6 , STRING7 , STRING8 , STRING9 , STRING10 , STRING11 , STRING12 , STRING13
 + STRING1 , STRING2 , STRING3 , STRING4 , STRING5 , STRING6 , STRING7 , STRING8 , STRING9 , STRING10 , STRING11 , STRING12
 } ;

 SINGLE


<nb> public final class CipherSuiteConverter {
 final String javaCipherSuiteTls = STRING0 + javaCipherSuiteSuffix ;

 SINGLE
 - final Map < String , String > p2j = new HashMap < > ( NUMBER0 ) ;
 + Map < String , String > p2j = new HashMap < > ( NUMBER0 ) ;
 p2j . put ( STRING1 , javaCipherSuiteSuffix ) ;
 p2j . put ( STRING2 , javaCipherSuiteSsl ) ;
 p2j . put ( STRING3 , javaCipherSuiteTls ) ;
 - o2j . putIfAbsent ( openSslCipherSuite , p2j ) ;
 + Map < String , String > p2jCurrent = o2j . putIfAbsent ( openSslCipherSuite , p2j ) ;
 + if ( p2jCurrent != null ) {
 + p2j = p2jCurrent ;
 + }

 SINGLE
 j2o . putIfAbsent ( javaCipherSuiteTls , openSslCipherSuite ) ;


<nb> public class ForCanBeForeachInspection extends StatementInspection {
 return false ;
 }
 final PsiExpression condition = forStatement . getCondition ( ) ;
 - if ( ! isArrayLengthComparison ( condition , indexVar ) ) {
 + if ( condition == null || ! isArrayLengthComparison ( condition , indexVar ) ) {
 return false ;
 }
 final PsiStatement update = forStatement . getUpdate ( ) ;
 <nb> public class ForCanBeForeachInspection extends StatementInspection {
 return false ;
 }

 - private static boolean isArrayLengthComparison ( PsiExpression condition ,
 + private static boolean isArrayLengthComparison ( @ notnull PsiExpression condition ,
 PsiLocalVariable var ) {
 final PsiExpression strippedCondition =
 ParenthesesUtils . stripParentheses ( condition ) ;
 <nb> final class ThumbnailManagerImpl extends ThumbnailManager implements Application
 public float getWeight ( ) {
 return NUMBER0 ;
 }
 +
 }
 }


<nb> public class OObjectSerializerHelper {
 if ( iMultiValue instanceof OLazyObjectMap < ? > ) {
 result = ( ( OLazyObjectMap < ? > ) iMultiValue ) . getUnderlying ( ) ;
 } else {
 + result = new HashMap < String , Object > ( ) ;
 for ( Entry < String , Object > entry : ( ( Map < String , Object > ) iMultiValue ) . entrySet ( ) ) {
 ( ( Map < String , Object > ) result ) . put ( entry . getKey ( ) ,
 typeToStream ( entry . getValue ( ) , linkedType , iEntityManager , iObj2RecHandler , db , iSaveOnlyDirty ) ) ;

Committed a patch by Luca Molino about marshalling of Maps on Detached objects


<nb> public class SetCharacterEncodingFilter implements Filter {
 COMMENT
 COMMENT
 COMMENT
 - protected String selectEncoding (
 - @ suppresswarnings ( STRING0 ) ServletRequest request ) {
 + protected String selectEncoding ( ServletRequest request ) {

 return ( this . encoding ) ;

Fix remaining warnings in webapps with recently documented settings


<nb> public abstract class NanoHTTPD {
 setAsyncRunner ( new DefaultAsyncRunner ( ) ) ;
 }

 - private static final void safeClose ( ServerSocket serverSocket ) {
 - if ( serverSocket != null ) {
 - try {
 - serverSocket . close ( ) ;
 - } catch ( IOException e ) {
 - }
 - }
 - }
 -
 - private static final void safeClose ( Socket socket ) {
 - if ( socket != null ) {
 - try {
 - socket . close ( ) ;
 - } catch ( IOException e ) {
 - }
 - }
 - }
 -
 private static final void safeClose ( Closeable closeable ) {
 if ( closeable != null ) {
 try {


<nb> public class StreetTransitLink extends AbstractEdge implements WalkableEdge {

 public TraverseResult traverseBack ( State s0 , TraverseOptions wo ) {
 State s1 = s0 . clone ( ) ;
 - if ( isBoarding ) {
 + if ( ! isBoarding ) {
 s1 . setJustBoarded ( true ) ;
 } else {
 if ( s0 . getJustBoarded ( ) ) {


<nb> public class FlowCursorIterator < TModel extends Model > implements ListIterator < TM
 if ( cursor != null ) {
 cursor . moveToPosition ( startingLocation - NUMBER0 ) ;
 reverseIndex = startingCount = cursor . getCount ( ) ;
 + reverseIndex -= startingLocation ;
 +
 + if ( reverseIndex < NUMBER1 ) {
 + reverseIndex = NUMBER1 ;
 + }
 }
 }

respect starting location in the reverseindex


<nb> public class Environment extends AbstractLifeCycle {
 COMMENT
 COMMENT
 COMMENT
 - public Object getJerseyProperty ( String name ) {
 - return config . getProperties ( ) . get ( name ) ;
 + public < T > T getJerseyProperty ( String name ) {
 + return ( T ) config . getProperties ( ) . get ( name ) ;
 }


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;

 COMMENT
 COMMENT
 <nb> class SlideExpandableListView extends ListView {
 this . setOnItemClickListener ( new OnItemClickListener ( ) {
 @ override
 public void onItemClick ( AdapterView < ? > adapterView , View view , int i , long l ) {
 - SlideExpandableListAdapter adapter = ( SlideExpandableListAdapter ) getAdapter ( ) ;
 - adapter . getExpandToggleButton ( view ) . performClick ( ) ;
 + if ( adapter != null ) {
 + adapter . getExpandToggleButton ( view ) . performClick ( ) ;
 + }
 }
 } ) ;
 }
 <nb> class SlideExpandableListView extends ListView {
 adapter . onRestoreInstanceState ( ss ) ;
 }
 }
 - }
 + }

Use own adapter not getAdapter it might have been wrapped by addHeaderView


<nb> public class ImmutableNameNotFoundException
 public void appendRemainingName ( Name name ) { }
 @ override
 public void setRemainingName ( Name name ) { }
 - public void setResolverName ( Name name ) { }
 + @ override
 + public void setResolvedName ( Name name ) { }
 @ override
 public void setRootCause ( Throwable e ) { }


<nb> public class ListenerList
 for ( ArrayList < IEventListener > list : priorities )
 {
 if ( list . remove ( listener ) )
 - {
 - rebuild = true ;
 - }
 + {
 + rebuild = true ;
 + }
 }
 }
 }


<nb> public interface TypeWriter < T > {
 currentLocalVariableLength = localVariableLength ;
 break ;
 default :
 - throw new IllegalStateException ( ) ;
 + throw new IllegalStateException ( STRING0 + type ) ;
 }
 }

 <nb> public class TypeWriterDefaultForInliningInitializationHandlerAppendingFrameWrit
 verifyZeroInteractions ( methodVisitor ) ;
 }

 + @ test ( expected = IllegalStateException . class )
 + public void testUnexpected ( ) throws Exception {
 + frameWriter . onFrame ( - NUMBER0 , NUMBER1 ) ;
 + }
 +
 @ test
 public void testObjectProperties ( ) throws Exception {
 ObjectPropertyAssertion . of ( TypeWriter . Default . ForInlining . InitializationHandler . Appending . FrameWriter . Active . class ) . applyBasic ( ) ;

Added error message and test for error on active frame handler


<nb> class ReaderPostRenderer {
 sbHtml . append ( STRING0 ) ;
 }

 - SINGLE
 - sbHtml . append ( STRING1 )
 - . append ( STRING2 ) . append ( pxToDp ( mResourceVars . fullSizeImageWidthPx ) ) . append ( STRING3 )
 - . append ( STRING4 ) ;
 + SINGLE
 + sbHtml . append ( STRING5 ) ;

 SINGLE
 sbHtml . append ( STRING6 )


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - public class DirectLogBuffer implements LogBuffer
 + class DirectLogBuffer implements LogBuffer
 {
 private final FileChannel fileChannel ;
 private final ByteBuffer buffer ;

 - public DirectLogBuffer ( FileChannel fileChannel , ByteBuffer buffer )
 + DirectLogBuffer ( FileChannel fileChannel , ByteBuffer buffer )
 {
 if ( fileChannel == null || buffer == null )
 {

Reverted access settings used when testing


<nb> public class WorkDuration implements Serializable {
 }

 static WorkDuration createFromMinutes ( long duration , int hoursInDay ) {
 - int days = ( ( Double ) ( duration / hoursInDay / NUMBER0 d ) ) . intValue ( ) ;
 + int days = ( ( Double ) ( ( double ) duration / hoursInDay / NUMBER0 d ) ) . intValue ( ) ;
 Long currentDurationInMinutes = duration - ( NUMBER1 * days * hoursInDay ) ;
 int hours = ( ( Double ) ( currentDurationInMinutes / NUMBER0 d ) ) . intValue ( ) ;
 currentDurationInMinutes = currentDurationInMinutes - ( NUMBER1 * hours ) ;
 <nb> public class RuleRegistration implements Startable {
 boolean changed = false ;

 CharacteristicDto characteristic = findCharacteristic ( characteristicDtos , def ) ;
 + SINGLE
 Integer characteristicId = characteristic != null ? characteristic . getId ( ) : null ;
 RemediationFunction remediationFunction = characteristic != null ? def . remediationFunction ( ) : null ;
 String remediationFactor = characteristic != null ? def . remediationFactor ( ) : null ;


<nb> public abstract class Structure {
 COMMENT
 mapper , and explicit field order will affect this information .
 COMMENT
 - private class LayoutInfo {
 + private static class LayoutInfo {
 int size = CALCULATE_SIZE ;
 int alignment = NUMBER0 ;
 Map fields = Collections . synchronizedMap ( new LinkedHashMap ( ) ) ;
 <nb> public abstract class Structure {
 }
 }

 - private class AutoAllocated extends Memory {
 + private static class AutoAllocated extends Memory {
 public AutoAllocated ( int size ) {
 super ( size ) ;
 SINGLE

FindBugs has some other suggestions about nested classes which could be static


<nb> public enum OGlobalConfiguration {
 final long maxProcessMemory = Runtime . getRuntime ( ) . maxMemory ( ) ;
 long mmapBestMemory = ( maxOsMemory - maxProcessMemory ) / NUMBER0 ;

 - FILE_MMAP_MAX_MEMORY . setValue ( NUMBER1 ) ;
 + FILE_MMAP_MAX_MEMORY . setValue ( mmapBestMemory ) ;
 FILE_MMAP_BLOCK_SIZE . setValue ( NUMBER2 ) ;
 MVRBTREE_NODE_PAGE_SIZE . setValue ( NUMBER3 ) ;
 } else {

Improved yet performance with huge datasets


<nb> public class TestLexerExec extends BaseTest {
 grammar . append ( STRING0 ) ;
 grammar . append ( STRING1 ) ;
 for ( int i = NUMBER0 ; i < NUMBER1 ; i ++ ) {
 - grammar . append ( STRING2 ) . append ( i ) . append ( STRING3 ) . append ( STRING2 ) . append ( i ) . append ( STRING4 ) ;
 + grammar . append ( STRING2 ) . append ( i ) . append ( STRING5 ) . append ( i ) . append ( STRING4 ) ;
 }

 String input = STRING6 ;

Fix test compile error by avoiding string literal assignments for the large lexer test


<nb> public static int hasheq ( Object o ) {
 if ( o instanceof Number )
 return Numbers . hasheq ( ( Number ) o ) ;
 if ( o instanceof String )
 - return Murmur3 . hashUnencodedChars ( ( CharSequence ) o ) ;
 + return Murmur3 . hashInt ( o . hashCode ( ) ) ;
 return o . hashCode ( ) ;
 }

murmur just string hashcode not string to take advantage of string s caching


<nb> public final class Es6ToEs3Converter implements NodeTraversal . Callback , HotSwapC
 private void visitClassMember (
 Node member , ClassDeclarationMetadata metadata ) {
 Node qualifiedMemberAccess = getQualifiedMemberAccess (
 - compiler , member ,
 + member ,
 NodeUtil . newQName ( compiler , metadata . fullClassName ) ,
 NodeUtil . newQName ( compiler , metadata . fullClassName + STRING0 ) ) ;
 Node method = member . getLastChild ( ) . detachFromParent ( ) ;
 <nb> public final class Es6ToEs3Converter implements NodeTraversal . Callback , HotSwapC
 COMMENT
 COMMENT
 COMMENT
 - private static Node getQualifiedMemberAccess ( AbstractCompiler compiler , Node member ,
 + private static Node getQualifiedMemberAccess ( Node member ,
 Node staticAccess , Node instanceAccess ) {
 Node context = member . isStaticMember ( ) ? staticAccess : instanceAccess ;
 context = context . cloneTree ( ) ;


<nb> import NAMESPACE ;
 COMMENT
 public class FileRotator {
 private static final String TAG = STRING0 ;
 - private static final boolean LOGD = true ;
 + private static final boolean LOGD = false ;

 private final File mBasePath ;
 private final String mPrefix ;
 <nb> import NAMESPACE ;
 COMMENT
 public class NetworkStatsRecorder {
 private static final String TAG = STRING1 ;
 - private static final boolean LOGD = true ;
 + private static final boolean LOGD = false ;
 private static final boolean LOGV = false ;

 private final FileRotator mRotator ;


<nb> public abstract class GenericFileEndpoint < T > extends ScheduledPollEndpoint imple
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public void setReadLockCheckInterval ( long readLockCheckInterval ) {
 this . readLockCheckInterval = readLockCheckInterval ;
 <nb> public abstract class GenericFileEndpoint < T > extends ScheduledPollEndpoint imple
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public void setReadLockTimeout ( long readLockTimeout ) {
 this . readLockTimeout = readLockTimeout ;

File endpoint should validate the read lock timeout interval options is configured correctly


<nb>
 COMMENT
 package NAMESPACE ;

 - import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class DexGraph implements IndexableGraph {
 final File db = new File ( fileName ) ;
 boolean create = ! db . exists ( ) ;
 this . db = db ;
 - DEX . Config cfg = new DEX . Config ( ) ;
 - cfg . setCacheMaxSize ( NUMBER0 ) ; SINGLE
 - dex = new DEX ( cfg ) ;
 + SINGLE
 + SINGLE
 + SINGLE
 + dex = new DEX ( ) ;
 gpool = ( create ? dex . create ( db ) : dex . open ( db ) ) ;
 session = gpool . newSession ( ) ;
 graph = session . getDbGraph ( ) ;

updated DexGraph constuctor to not allow for all memory to be consumed


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

fixed checkstyle issue by removing the unused import


<nb> public class AnalyticsTrackerMixpanel implements AnalyticsTracker . Tracker {
 private EnumMap < AnalyticsTracker . Stat , JSONObject > mAggregatedProperties ;

 private static final String SESSION_COUNT = STRING0 ;
 - private static final String MIXPANEL_PLATFORM = STRING1 ;
 + private static final String MIXPANEL_PLATFORM = STRING2 ;
 private static final String MIXPANEL_SESSION_COUNT = STRING3 ;
 private static final String DOTCOM_USER = STRING4 ;
 private static final String JETPACK_USER = STRING5 ;


<nb> public class RCompletionManager implements CompletionManager
 {
 left = STRING0 ;
 }
 - else if ( splat . length == NUMBER0 )
 - {
 - left = splat [ NUMBER1 ] ;
 - }
 else
 {
 left = splat [ NUMBER1 ] ;


<nb> package NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 <nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - public class PLCBusGenericBindingProvider extends
 - AbstractGenericBindingProvider implements PLCBusBindingProvider ,
 - AutoUpdateBindingProvider {
 + public class PLCBusGenericBindingProvider extends AbstractGenericBindingProvider implements PLCBusBindingProvider {

 @ override
 public String getBindingType ( ) {
 <nb> public class PLCBusGenericBindingProvider extends
 PLCBusBindingConfig config = new PLCBusBindingConfig ( bindingConfig ) ;
 addBindingConfig ( item , config ) ;
 }
 -
 - @ override
 - public Boolean autoUpdate ( String itemName ) {
 - return true ;
 - }
 -
 +
 @ override
 public PLCBusBindingConfig getConfigFor ( String itemName ) {
 return ( PLCBusBindingConfig ) bindingConfigs . get ( itemName ) ;

removed reference to AutoUpdateBindingProvider since it constantly returns true which is the default as well


<nb> public class JCVideoPlayer extends FrameLayout implements View . OnClickListener ,
 return ;
 }
 if ( videoEvents . type == VideoEvents . VE_PREPARED ) {
 + if ( CURRENT_STATE != CURRENT_STATE_PREPAREING ) return ;
 JCMediaPlayer . intance ( ) . mediaPlayer . setDisplay ( surfaceHolder ) ;
 JCMediaPlayer . intance ( ) . mediaPlayer . start ( ) ;
 pbLoading . setVisibility ( View . INVISIBLE ) ;


<nb> import NAMESPACE ;

 import NAMESPACE ;

 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public final class UpgradeService {
 Preferences . clear ( AstridPreferences . P_UPGRADE_FROM ) ;
 StringBuilder changeLog = new StringBuilder ( ) ;

 - if ( from >= V3_9_1_1 && from < V3_9_2 ) {
 + if ( from < V3_9_2 ) {
 newVersionString ( changeLog , STRING0 , new String [ ] {
 STRING1 ,
 STRING2 ,


<nb> public class RSConnectDeploy extends Composite

 public void showAppInfo ( RSConnectApplicationInfo info )
 {
 - if ( info == null )
 + if ( info != null )
 {
 - appInfoPanel_ . setVisible ( false ) ;
 - newAppPanel_ . setVisible ( true ) ;
 - return ;
 + urlAnchor_ . setText ( info . getUrl ( ) ) ;
 + urlAnchor_ . setHref ( info . getUrl ( ) ) ;
 }
 -
 - urlAnchor_ . setText ( info . getUrl ( ) ) ;
 - urlAnchor_ . setHref ( info . getUrl ( ) ) ;
 appInfoPanel_ . setVisible ( true ) ;
 appDetailsPanel_ . setVisible ( true ) ;
 newAppPanel_ . setVisible ( false ) ;
 <nb> public class RSConnectDeploy extends Composite
 SINGLE
 SINGLE
 appProgressPanel_ . setVisible ( false ) ;
 + showAppInfo ( null ) ;
 }
 } ) ;
 }

tolerate server errors when looking up application URL


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - @ suppresswarnings ( STRING0 )
 @ jsonserialize (
 include = JsonSerialize . Inclusion . NON_NULL
 )


<nb> public class DelegatingBindingTrace implements BindingTrace {
 private final BindingContext bindingContext = new BindingContext ( ) {
 @ override
 public Collection < Diagnostic > getDiagnostics ( ) {
 - throw new UnsupportedOperationException ( ) ; SINGLE
 + return diagnostics ;
 }

 @ override
 <nb> public class DelegatingBindingTrace implements BindingTrace {
 }

 @ override
 + @ notnull
 public BindingContext getBindingContext ( ) {
 return bindingContext ;
 }

Implemented getDiagnostics in BindingContext returned by DelegatingBindingTrace


<nb> public abstract class QuadTree < G extends QuadTree . GeometricObject > {

 private boolean insertIntoChildren ( AABB b ) {
 SINGLE
 - if ( northWest . insert ( b ) ) return true ;
 - if ( northEast . insert ( b ) ) return true ;
 - if ( southWest . insert ( b ) ) return true ;
 - if ( southEast . insert ( b ) ) return true ;
 + if ( northWest . aabb . insideThis ( b ) && northWest . insert ( b ) ) return true ;
 + if ( northEast . aabb . insideThis ( b ) && northEast . insert ( b ) ) return true ;
 + if ( southWest . aabb . insideThis ( b ) && southWest . insert ( b ) ) return true ;
 + if ( southEast . aabb . insideThis ( b ) && southEast . insert ( b ) ) return true ;
 return false ;
 }

Using a slightly move efficient insert method for rectangle quadtree


<nb> class KeyManager {
 for ( Object part : ( ( MultiKey ) key ) . getKeys ( ) ) {
 setUp ( part ) ;
 }
 + ensureNode ( parent , key ) . uses ++ ;
 } else if ( key instanceof TreeKey ) {
 TreeKey treeKey = ( TreeKey ) key ;
 List < ? > elements = treeKey . getKeyPath ( ) ;
 <nb> class KeyManager {

 void tearDown ( Object key ) {
 if ( key instanceof MultiKey ) {
 + decrementAndMaybeRemoveKey ( key ) ;
 final List < Object > parts = ( ( MultiKey ) key ) . getKeys ( ) ;
 final int count = parts . size ( ) ;
 for ( int i = count - NUMBER0 ; i >= NUMBER1 ; i -- ) {
 - tearDown ( parts . get ( count ) ) ;
 + tearDown ( parts . get ( i ) ) ;
 }
 } else if ( key instanceof TreeKey ) {
 TreeKey treeKey = ( TreeKey ) key ;


<nb> public class Validator
 sb . append ( STRING0 + queryPair . getControl ( ) . getSchema ( ) + STRING1 ) ;
 sb . append ( STRING2 ) ;
 }
 + else {
 + sb . append ( STRING3 ) ;
 + }
 return sb . toString ( ) ;
 }

Print SKIPPED instead of an empty message when skipping a query


<nb> public class MessageList

 if ( mFolderName != null )
 {
 - mController . listLocalMessagesSynchronous ( mAccount , mFolderName , mAdapter . mListener ) ;
 + mController . listLocalMessages ( mAccount , mFolderName , mAdapter . mListener ) ;
 mController . notifyAccountCancel ( this , mAccount ) ;

 MessagingController . getInstance ( getApplication ( ) ) . notifyAccountCancel ( this , mAccount ) ;

With all the other performance work we ve been doing getting a message


<nb> public final class PersistentPeriodicImportJob
 }
 } ;

 - public static final Function < PersistentPeriodicImportJob , Long > jobIdGetter ( )
 + public static Function < PersistentPeriodicImportJob , Long > jobIdGetter ( )
 {
 return GET_JOB_ID ;
 }


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 public final class StringsResourceTranslator {

 - private static final String API_KEY = STRING0 ;
 + private static final String API_KEY = System . getProperty ( STRING1 ) ;

 private static final Charset UTF8 = Charset . forName ( STRING2 ) ;
 private static final Pattern ENTRY_PATTERN = Pattern . compile ( STRING3 ) ;
 <nb> public final class StringsResourceTranslator {
 }
 System . out . println ( STRING4 + english ) ;

 + if ( API_KEY == null ) {
 + throw new IllegalArgumentException ( STRING5 ) ;
 + }
 +
 URL translateURL = new URL (
 STRING6 + API_KEY + STRING7 +
 URLEncoder . encode ( english , STRING2 ) +

Make API settable in the command line as system property


<nb> public class SubstitutionStepdefs {

 @ then ( STRING0 )
 public void I_should_receive_an_email_with_the_body ( String body ) throws Throwable {
 - String expected = String . format ( STRING1 +
 - STRING2 +
 + String expected = String . format ( STRING3 +
 + STRING4 +
 STRING5 , name , role , details ) ;
 assertEquals ( expected , body ) ;
 }

Replace n with n so that tests work on Windows


<nb> import NAMESPACE ;
 public enum DiskCacheStrategy {

 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 ALL ( true COMMENT , true COMMENT ) {
 @ override
 public boolean isDataCacheable ( DataSource dataSource ) {
 - return dataSource != DataSource . DATA_DISK_CACHE && dataSource != DataSource . MEMORY_CACHE ;
 + return dataSource == DataSource . REMOTE ;
 }

 @ override

Don t save local data using DiskCacheStrategy ALL


<nb> package NAMESPACE ;
 COMMENT
 public class ToDoConstants {

 - SINGLE
 + SINGLE

 SINGLE
 SINGLE
 <nb> public class ToDoConstants {
 SINGLE
 SINGLE
 SINGLE
 + SINGLE

 SINGLE


<nb> public class PercentageDrawable extends Drawable {
 private float mTargetValue ;

 private String mText ;
 + private float mTextX , mTextY ;

 private boolean mEnabled ;
 private boolean mPressed ;
 <nb> public class PercentageDrawable extends Drawable {
 mForegroundCirclePaint ) ;

 if ( mText != null ) {
 + final float width = mTextPaint . measureText ( mText ) ;
 mTextPaint . getTextBounds ( mText , NUMBER0 , mText . length ( ) , mTextBounds ) ;
 canvas . drawText ( mText ,
 - mBounds . centerX ( ) - ( mTextBounds . width ( ) / FLOAT0 ) ,
 + mBounds . centerX ( ) - ( width / FLOAT0 ) ,
 mBounds . centerY ( ) + ( mTextBounds . height ( ) / FLOAT0 ) ,
 mTextPaint ) ;
 }


<nb> public class ConfusionTask extends MRTask {
 if ( _classWt != null ) SINGLE
 for ( int v = NUMBER0 ; v < _N ; v ++ ) vi [ v ] = ( int ) ( vi [ v ] * _classWt [ v ] ) ;
 int result = Model . getPrediction ( vi , ties , row ) ; SINGLE
 - if ( vi [ result ] == NUMBER0 ) { cm . _skippedRows ++ ; System . err . println ( STRING0 + row + STRING1 + result ) ; continue ; } SINGLE
 + if ( vi [ result ] == NUMBER0 ) { cm . _skippedRows ++ ; COMMENT continue ; } SINGLE

 int cclass = alignDataIdx ( ( int ) _data . data ( bits , row , _classcol ) - cmin ) ;
 assert NUMBER0 <= cclass && cclass < _N : ( STRING2 + cclass + STRING3 + _N ) ;


<nb> public class AppBarLayout extends LinearLayout {
 }

 @ override
 - public boolean onStartNestedScroll ( CoordinatorLayout coordinatorLayout , AppBarLayout child ,
 + public boolean onStartNestedScroll ( CoordinatorLayout parent , AppBarLayout child ,
 View directTargetChild , View target , int nestedScrollAxes ) {
 SINGLE
 SINGLE
 final boolean started = ( nestedScrollAxes & ViewCompat . SCROLL_AXIS_VERTICAL ) != NUMBER0
 && child . hasScrollableChildren ( )
 - && coordinatorLayout . getHeight ( ) - target . getHeight ( ) <= child . getHeight ( ) ;
 + && parent . getHeight ( ) - directTargetChild . getHeight ( ) <= child . getHeight ( ) ;

 if ( started && mAnimator != null ) {
 SINGLE

Fix AppBarLayout scrolling when scrolling view is small


<nb> public class TestSharding extends AbstractServerClusterTest {
 for ( OrientVertex v : result ) {
 System . out . println ( STRING0 + v . getRecord ( ) ) ;

 - Assert . assertEquals ( STRING1 + server , ( Long ) totalAmount , ( Long ) v . getProperty ( STRING2 ) ) ;
 + SINGLE
 + SINGLE

 count ++ ;
 }


<nb> public class GwtAtmosphereResourceImpl implements GwtAtmosphereResource {
 COMMENT
 COMMENT
 void resumeAfterDeath ( ) {
 - atmosphereHandler . execute ( new Runnable ( ) {
 - @ override
 - public void run ( ) {
 - atmResource . resume ( ) ;
 - }
 - } ) ;
 + if ( atmosphereHandler != null ) {
 + atmosphereHandler . execute ( new Runnable ( ) {
 + @ override
 + public void run ( ) {
 + atmResource . resume ( ) ;
 + }
 + } ) ;
 + }
 }

 @ override


<nb> import NAMESPACE ;
 COMMENT
 public class RuntimeMetaData {
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public static final String VERSION = STRING0 ;
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public class RuntimeMetaData {
 }

 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - public static final String VERSION = STRING0 ;
 -
 - COMMENT
 COMMENT
 COMMENT
 COMMENT

Move RuntimeMetaData VERSION field to the top of the source file


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 public class LuceneQuery implements SimpleQuery < LuceneQuery > , SimpleProjectable < Document > {
 - public static final boolean DEFAULT = false ;
 - public static final boolean LOWER_CASE = true ;
 -
 +
 private final QueryMixin < LuceneQuery > queryMixin ;

 private final LuceneSerializer serializer ;
 <nb> public class LuceneQueryTest {
 writer . close ( ) ;

 searcher = new IndexSearcher ( idx ) ;
 - query = new LuceneQuery ( LuceneQuery . LOWER_CASE , searcher ) ;
 + query = new LuceneQuery ( true , searcher ) ;
 }

 @ after


<nb> final class NounInflector {
 return word ;
 }
 SINGLE
 - return String . valueOf ( Character . toLowerCase ( first ) ) + word . substring ( NUMBER0 ) ;
 + StringBuilder sb = new StringBuilder ( word . length ( ) ) ;
 + sb . append ( Character . toLowerCase ( first ) ) ;
 + sb . append ( word . substring ( NUMBER0 ) ) ;
 + return sb . toString ( ) ;
 }

 COMMENT


<nb> public class SpargelConnectedComponentsITCase extends JavaProgramTestBase {
 DataSet < Tuple2 < Long , Long > > initialVertices = vertexIds . map ( new IdAssigner ( ) ) ;
 DataSet < Tuple2 < Long , Long > > result = initialVertices . runOperation ( VertexCentricIteration . withPlainEdges ( edges , new CCUpdater ( ) , new CCMessager ( ) , NUMBER0 ) ) ;

 - result . print ( ) ;
 + result . writeAsCsv ( resultPath , STRING0 , STRING1 ) ;
 env . execute ( STRING2 ) ;
 }

SpargelConnectedComponentsITCase properly verifies results


<nb> public abstract class ORecordSchemaAwareAbstract < T > extends ORecordAbstract < T > i
 throw new OValidationException ( STRING0 + p . getFullName ( ) + STRING1 + p . getType ( )
 + STRING2 + fieldValue ) ;
 else if ( fieldValue instanceof OIdentifiable ) {
 - if ( ! ( ( OIdentifiable ) fieldValue ) . getIdentity ( ) . isValid ( ) )
 + if ( ( ( OIdentifiable ) fieldValue ) . getIdentity ( ) . isValid ( ) )
 throw new OValidationException ( STRING0 + p . getFullName ( ) + STRING1 + p . getType ( )
 + STRING3 + fieldValue ) ;


<nb> public class NotificationManagerService extends SystemService {
 }
 if ( notification . actions != null ) {
 for ( Notification . Action action : notification . actions ) {
 + if ( action . actionIntent == null ) {
 + continue ;
 + }
 am . setPendingIntentWhitelistDuration ( action . actionIntent . getTarget ( ) , duration ) ;
 setPendingIntentWhitelistDuration ( am , duration , action . getExtras ( ) ) ;
 final RemoteInput [ ] remoteInputs = action . getRemoteInputs ( ) ;


<nb> public class RabbitMQProducer extends DefaultProducer {
 COMMENT
 COMMENT
 COMMENT
 - private static interface ChannelCallback < T > {
 + private interface ChannelCallback < T > {
 T doWithChannel ( Channel channel ) throws Exception ;
 }


<nb> public class DependencyResolver {
 JavaScopes . SYSTEM ) ;

 private final String [ ] EXCLUSIONS = new String [ ] {
 - STRING0
 + STRING0 ,
 + STRING1 ,
 + STRING2 ,
 + STRING3 ,
 + STRING4
 } ;

Add scala compiler and zeppelin related library to dependency exclusion list


<nb> public class UndertowDeploymentService implements Service < UndertowDeploymentServ
 if ( file == null ) {
 return ;
 }
 - for ( File f : file . listFiles ( ) ) {
 + File [ ] files = file . listFiles ( ) ;
 + if ( files == null ) {
 + return ;
 + }
 + for ( File f : files ) {
 recursiveDelete ( f ) ;
 }
 if ( ! file . delete ( ) ) {


<nb> public class GraphServiceImpl implements GraphService {
 LOG . error ( STRING0 , routerId ) ;
 throw new GraphNotFoundException ( ) ;
 } else {
 - return graphSources . get ( routerId ) . getGraph ( ) ;
 + Graph graph = graphSources . get ( routerId ) . getGraph ( ) ;
 + if ( graph == null ) {
 + evictGraph ( routerId ) ;
 + throw new GraphNotFoundException ( ) ;
 + }
 + return graph ;
 }
 }
 }


<nb> public class AboutPanel extends JPanel {
 private static final long serialVersionUID = NUMBER0 ;
 private static final String PRODUCT = Constant . PROGRAM_NAME ;
 private static final String VERSION = STRING0 + Constant . PROGRAM_VERSION ;
 - private static final String COPYRIGHT = STRING1 ;
 + private static final String COPYRIGHT = STRING2 ;
 private static final String LICENSE_DETAIL = STRING3
 + STRING4
 + STRING5


<nb> public class SynchronizeFolderOperation extends RemoteOperation {
 sendLocalBroadcast ( EVENT_SINGLE_FOLDER_CONTENTS_SYNCED , mLocalFolder . getRemotePath ( ) , result ) ;
 }

 - if ( result . isSuccess ( ) && mIsShareSupported ) {
 - RemoteOperationResult shareResult = refreshSharesForFolder ( client ) ;
 - if ( shareResult . getCode ( ) != ResultCode . FILE_NOT_FOUND ) {
 - result = shareResult ;
 - } SINGLE
 + if ( result . isSuccess ( ) && mIsShareSupported && ! mSyncFullAccount ) {
 + refreshSharesForFolder ( client ) ; SINGLE
 }

 if ( ! mSyncFullAccount ) {

Recursive refresh of folders in full synchronization does not retrieve information about shares


<nb> public enum IdType
 LABEL_TOKEN ( false ) ,
 LABEL_TOKEN_NAME ( false ) ,
 NEOSTORE_BLOCK ( false ) ,
 - SCHEMA ( NUMBER0 , true ) ,
 + SCHEMA ( NUMBER0 , false ) ,
 NODE_LABELS ( NUMBER0 , true ) ;

 private final long max ;

Fix recovery issue for constraint indexes


<nb> public class BlazeCommandUtils {
 throw new IllegalStateException ( STRING0 + topic + STRING1 ) ;
 }

 - return template .
 - replace ( STRING2 , Constants . PRODUCT_NAME ) .
 - replace ( STRING3 , topic ) .
 - replace ( STRING4 , parser . describeOptions ( categoryDescriptions , helpVerbosity ) ) .
 - trim ( )
 + String optionStr =
 + parser
 + . describeOptions ( categoryDescriptions , helpVerbosity )
 + . replace ( STRING2 , Constants . PRODUCT_NAME ) ;
 + return template
 + . replace ( STRING2 , Constants . PRODUCT_NAME )
 + . replace ( STRING3 , topic )
 + . replace ( STRING4 , optionStr )
 + . trim ( )
 + STRING5
 + ( helpVerbosity == OptionsParser . HelpVerbosity . MEDIUM
 - ? STRING6
 - : STRING7 ) ;
 + ? STRING6
 + : STRING7 ) ;
 }

 COMMENT

Replace product in option help texts too


<nb> public class MapActivityActions implements DialogProvider {
 SINGLE
 if ( getTargets ( ) . getPointToNavigate ( ) != null ) {
 optionsMenuHelper . item ( R . string . target_points ) . icons ( R . drawable . ic_action_flage_dark ,
 - R . drawable . ic_action_pin_light )
 + R . drawable . ic_action_flage_light )
 . listen ( new OnContextMenuClick ( ) {
 @ override
 public void onContextMenuClick ( int itemId , int pos , boolean isChecked , DialogInterface dialog ) {


<nb> public class SmartNullReturnValues implements ReturnValues {

 public Object intercept ( Object obj , Method method , Object [ ] args , MethodProxy proxy ) throws Throwable {
 if ( Invocation . isToString ( method ) ) {
 - return STRING0 + invocation . getMethod ( ) . getName ( ) + STRING1 ;
 + return STRING2 + invocation . getMethod ( ) . getName ( ) + STRING1 ;
 }
 throw new SmartNullPointerException ( STRING3 , whenCreated ) ;
 }


<nb> public abstract class DiskCacheStrategy {
 public static final DiskCacheStrategy RESOURCE = new DiskCacheStrategy ( ) {
 @ override
 public boolean isDataCacheable ( DataSource dataSource ) {
 - return true ;
 + return false ;
 }

 @ override


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - public class Point extends ArrayPath < Double > {
 + public class Point extends ArrayPath < Double [ ] , Double > {

 private static final long serialVersionUID = NUMBER0 ;


<nb>

 package NAMESPACE ;

 + import static NAMESPACE ;
 + import static NAMESPACE ;
 +
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;

 - COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 public class IsSubtypeOf < T extends Tree > extends AbstractTypeMatcher < T > {

 public IsSubtypeOf ( Supplier < Type > typeToCompareSupplier ) {
 <nb> public class IsSubtypeOf < T extends Tree > extends AbstractTypeMatcher < T > {

 @ override
 public boolean matches ( T tree , VisitorState state ) {
 - Types types = state . getTypes ( ) ;
 - Type typeToCompare = typeToCompareSupplier . get ( state ) ;
 - return ( typeToCompare != null &&
 - types . isSubtype ( ( ( JCTree ) tree ) . type , types . erasure ( typeToCompare ) ) ) ;
 + return isSubtype ( getType ( tree ) , typeToCompareSupplier . get ( state ) , state ) ;
 }
 }


<nb> public class PyFileEvaluator {
 }
 return Collections . emptyList ( ) ;
 }
 +
 + public Set < PyFile > getVisitedFiles ( ) {
 + return myVisitedFiles ;
 + }
 }

use proper CachedValue for caching of Django settings values


<nb> import NAMESPACE ;
 COMMENT
 public class IndexStoreConfig implements NodeSettingsService . Listener {

 - COMMENT
 - COMMENT
 - private static final ByteSizeValue DEFAULT_THROTTLE = new ByteSizeValue ( NUMBER0 , ByteSizeUnit . MB ) ;
 -
 COMMENT
 COMMENT
 COMMENT
 <nb> public class IndexStoreConfig implements NodeSettingsService . Listener {
 SINGLE
 this . rateLimitingType = settings . get ( STRING0 , StoreRateLimiting . Type . NONE . name ( ) ) ;
 rateLimiting . setType ( rateLimitingType ) ;
 - this . rateLimitingThrottle = settings . getAsBytesSize ( STRING1 , DEFAULT_THROTTLE ) ;
 + this . rateLimitingThrottle = settings . getAsBytesSize ( STRING1 , new ByteSizeValue ( NUMBER1 ) ) ;
 rateLimiting . setMaxRate ( rateLimitingThrottle ) ;
 logger . debug ( STRING2 , rateLimitingType , rateLimitingThrottle ) ;
 }

remove confusing private default constant


<nb> public class DisambiguationPatternRule {
 int skipShiftTotal = NUMBER0 ;

 int firstMatchToken = - NUMBER1 ;
 - int lastMatchToken = - NUMBER1 ;
 final int patternSize = patternElements . size ( ) ;
 Element elem = null , prevElement = null ;
 final boolean startWithSentStart = patternElements . get ( NUMBER0 ) . isSentStart ( ) ;
 <nb> public class DisambiguationPatternRule {
 allElementsMatch = skipMatch ;
 if ( skipMatch ) {
 prevSkipNext = skipNext ;
 - matchingTokens ++ ;
 - lastMatchToken = matchPos ;
 + matchingTokens ++ ;
 if ( firstMatchToken == - NUMBER1 ) {
 firstMatchToken = matchPos ;
 }
 <nb> public class DisambiguationPatternRule {

 }
 }
 - firstMatchToken = - NUMBER1 ;
 - lastMatchToken = - NUMBER1 ;
 + firstMatchToken = - NUMBER1 ;
 skipShiftTotal = NUMBER0 ;
 language . getUnifier ( ) . reset ( ) ;
 inUnification = false ;


<nb> public abstract class AbstractServer extends UnicastRemoteObject
 catch ( NoSuchObjectException e )
 {
 SINGLE
 - System . out . println ( STRING0 ) ;
 + SINGLE
 }
 }

Removed a System out println when shutdown failed


<nb> public class TfsShell implements Closeable {
 command . run ( args ) ;
 return NUMBER0 ;
 } catch ( IOException e ) {
 - System . out . println ( e . getMessage ( ) ) ;
 + e . printStackTrace ( ) ;
 return - NUMBER1 ;
 }
 }

Print stacktraces when exceptions are thrown in tfs shell


<nb> final class BluetoothHealthProfileHandler {
 int newDeviceState ) {
 mHealthDevices . put ( device , newDeviceState ) ;
 mBluetoothService . sendConnectionStateChange ( device , BluetoothProfile . HEALTH ,
 - prevDeviceState , newDeviceState ) ;
 + newDeviceState , prevDeviceState ) ;
 }

 COMMENT

Fix sendConnection State intent api params


<nb> public class LoggingHandler extends ChannelDuplexHandler {
 }

 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Improve documentation for the two args formatter in LoggingHandler


<nb> public class HttpServerCodecTest {
 Assert . assertEquals ( offeredContentLength , totalBytesPolled ) ;
 }

 - private ByteBuf prepareDataChunk ( int size ) {
 + private static ByteBuf prepareDataChunk ( int size ) {
 StringBuilder sb = new StringBuilder ( ) ;
 for ( int i = NUMBER0 ; i < size ; ++ i ) {
 sb . append ( STRING0 ) ;

Add static modifier where static member is just enough


<nb> public class RecyclerView extends ViewGroup {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Corrected onChange animation documentation


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;

 public class HpackEncoder
 {
 <nb> public class HpackEncoder
 return _context ;
 }

 - COMMENT
 public ByteBufferPool . Lease encode ( HttpFields fields )
 {
 - return new ByteBufferPool . Lease ( _byteBufferPool ) ;
 + Lease lease = new ByteBufferPool . Lease ( _byteBufferPool ) ;
 + ByteBuffer buffer = _byteBufferPool . acquire ( NUMBER0 * NUMBER1 , false ) ; SINGLE
 + lease . add ( buffer , true ) ;
 + SINGLE
 + encode ( buffer , fields ) ;
 + return lease ;
 }
 - COMMENT

 public void encode ( ByteBuffer buffer , HttpFields fields )
 {


<nb> public class Platform {
 } catch ( SecurityException se ) {
 SINGLE
 SINGLE
 - throw new IOException ( STRING0 , se ) ;
 + IOException ioException = new IOException ( STRING0 ) ;
 + ioException . initCause ( se ) ;
 + throw ioException ;
 }
 }

 -
 @ override public void enableTlsExtensions ( SSLSocket socket , String uriHost ) {
 super . enableTlsExtensions ( socket , uriHost ) ;
 if ( openSslSocketClass . isInstance ( socket ) ) {

Don t use IOException APIs not in Gingerbread


<nb> public class ImapResponseParser {
 response . add ( firstToken ) ;

 if ( isStatusResponse ( firstToken ) ) {
 - parseStatusResponse ( response ) ;
 + parseResponseText ( response ) ;
 } else {
 Object token ;
 while ( ( token = readToken ( response ) ) != null ) {
 <nb> public class ImapResponseParser {
 response . mCompleted = true ;
 }

 - void parseStatusResponse ( ImapResponse parent ) throws IOException {
 + void parseResponseText ( ImapResponse parent ) throws IOException {

 skipIfSpace ( ) ;

Renamed parseStatusResponse to parseResponseText


<nb> public abstract class ColoredListCellRenderer extends SimpleColoredComponent imp
 public Component getListCellRendererComponent ( JList list , Object value , int index , boolean selected , boolean hasFocus ) {
 clear ( ) ;

 + setFont ( list . getFont ( ) ) ;
 mySelected = selected ;
 myForeground = list . getForeground ( ) ;
 mySelectionForeground = list . getSelectionForeground ( ) ;
 <nb> public class Switcher extends AnAction implements DumbAware {
 final Component c = super . getListCellRendererComponent ( list , value , index , selected , hasFocus ) ;
 final Color bg = UIUtil . getListBackground ( ) ;
 final Color fg = UIUtil . getListForeground ( ) ;
 + myLabel . setFont ( list . getFont ( ) ) ;
 myLabel . setForeground ( open ? fg : bg ) ;

 myPanel . removeAll ( ) ;

Renderer must respect font installed in JList


<nb> public class FormatPostfixTemplate extends StringBasedPostfixTemplate {
 }

 @ override
 - protected boolean shouldRemoveParent ( ) {
 - return false ;
 + protected PsiElement getElementToRemove ( PsiElement expr ) {
 + return expr ;
 }
 }

FormatPostfixTemplate replace deprecated usage of method shouldRemoveParent with usage of getElementToRemove


<nb>
 COMMENT
 package NAMESPACE ;

 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public abstract class ShowChangeMarkerAction extends AbstractVcsAction {
 protected void update ( VcsContext context , Presentation presentation ) {
 boolean active = isActive ( context ) ;
 presentation . setEnabled ( active ) ;
 - presentation . setVisible ( active ) ;
 + presentation . setVisible ( context . getEditor ( ) != null ) ;
 }

Only hide default next prev change actions if there s no text editor in sight


<nb> import NAMESPACE ;
 import NAMESPACE ;

 public class DefaultCacheLockingManager implements CacheLockingManager {
 - public static final int CACHE_LAYOUT_VERSION = NUMBER0 ;
 + public static final int CACHE_LAYOUT_VERSION = NUMBER1 ;
 private final PersistentCache cache ;

 public DefaultCacheLockingManager ( CacheRepository cacheRepository ) {

Bump cache version due to change of DefaultExternalResourceMetaData


<nb> public class PostgresDatabase extends AbstractDatabase {

 @ override
 protected String getDefaultDatabaseSchemaName ( ) throws DatabaseException {
 -
 + if ( getDefaultSchemaName ( ) != null ) {
 + return getDefaultSchemaName ( ) ;
 + }
 if ( defaultDatabaseSchemaName == null ) {
 try {
 List < String > searchPaths = getSearchPaths ( ) ;


<nb> public class WordPressDB {
 public static final String COLUMN_NAME_VIDEO_PRESS_SHORTCODE = STRING0 ;
 public static final String COLUMN_NAME_UPLOAD_STATE = STRING1 ;

 - private static final int DATABASE_VERSION = NUMBER0 ;
 + private static final int DATABASE_VERSION = NUMBER1 ;

 private static final String CREATE_TABLE_BLOGS = STRING2
 + STRING3 ;
 <nb> public class WordPressDB {
 SINGLE
 private static final String ADD_BLOGS_HIDDEN_FLAG = STRING4 ;

 + SINGLE
 + private static final String ADD_BLOGS_PLAN_ID = STRING5 ;
 +
 SINGLE
 private static final String DEPRECATED_WPCOM_USERNAME_PREFERENCE = STRING6 ;
 private static final String DEPRECATED_ACCESS_TOKEN_PREFERENCE = STRING7 ;
 <nb> public class WordPressDB {
 case NUMBER2 :
 AccountTable . migrationAddAccountSettingsFields ( db ) ;
 currentVersion ++ ;
 + case NUMBER0 :
 + db . execSQL ( ADD_BLOGS_PLAN_ID ) ;
 + currentVersion ++ ;
 }
 db . setVersion ( DATABASE_VERSION ) ;
 }


<nb> public class ShareUtils {
 final ServiceManager manager = new ServiceManager ( ) ;
 manager . setApiKey ( traktApiKey ) ;
 manager . setAuthentication ( username , passwordHash ) ;
 - SINGLE
 manager . setUseSsl ( true ) ;

 Response response = null ;
 <nb> public class Utils {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 public static long convertToFakeTime ( long time , SharedPreferences prefs , boolean isCurrentTime ) {


<nb>
 package NAMESPACE ;

 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public final class LocalizedMessage
 return false ;
 }

 - SINGLE
 + if ( ! Arrays . equals ( mArgs , localizedMessage . mArgs ) ) {
 + return false ;
 + }
 + SINGLE

 - SINGLE
 - SINGLE
 + SINGLE

 return true ;
 }
 <nb> public final class LocalizedMessage
 result = mLineNo ;
 result = NUMBER0 * result + mColNo ;
 result = NUMBER0 * result + mKey . hashCode ( ) ;
 + for ( int i = NUMBER1 ; i < mArgs . length ; i ++ ) {
 + result = NUMBER0 * result + mArgs [ i ] . hashCode ( ) ;
 + }
 return result ;
 }

Compare mArgs in order to handle checks such as


<nb> public final class DefaultPassConfig extends PassConfig {
 SINGLE
 SINGLE
 SINGLE
 - if ( options . getLanguageIn ( ) . isEs6OrHigher ( ) ) {
 + if ( options . getLanguageIn ( ) . isEs6OrHigher ( ) && ! options . skipTranspilationAndCrash ) {
 checks . add ( es6RewriteArrowFunction ) ;
 checks . add ( es6RenameVariablesInParamLists ) ;
 checks . add ( es6SplitVariableDeclarations ) ;

Add missing check to disable other transpilation passes


<nb> public class TextEditingTarget implements
 @ handler
 void onCommentUncomment ( )
 {
 - if ( fileType_ . isCpp ( ) )
 - doCommentUncomment ( STRING0 ) ;
 - else if ( isCursorInTexMode ( ) )
 + if ( isCursorInTexMode ( ) )
 doCommentUncomment ( STRING1 ) ;
 else if ( isCursorInRMode ( ) )
 doCommentUncomment ( STRING2 ) ;
 + else if ( fileType_ . isCpp ( ) )
 + doCommentUncomment ( STRING0 ) ;
 +
 }

 private void doCommentUncomment ( String c )


<nb> public class ViewerPresenter extends BasePresenter
 {
 manageCommands ( true ) ;
 display_ . bringToFront ( ) ;
 + if ( Desktop . isDesktop ( ) )
 + Desktop . getFrame ( ) . setViewerUrl ( event . getParams ( ) . getUrl ( ) ) ;
 display_ . previewShiny ( event . getParams ( ) ) ;
 runningShinyAppParams_ = event . getParams ( ) ;
 }

set viewer URL when previewing Shiny in the viewer pane


<nb> public class TtsService extends Service implements OnCompletionListener {
 if ( synthAvailable ) {
 synthesizerLock . unlock ( ) ;
 }
 + processSpeechQueue ( ) ;
 }
 }
 }

Fixing the synth function so that queue processing continues


<nb> public class DroidGap extends PhonegapActivity {
 COMMENT
 protected void onPause ( ) {
 super . onPause ( ) ;
 -
 + SINGLE
 + this . appView . loadUrl ( STRING0 ) ;
 +
 SINGLE
 if ( ! this . keepRunning ) {

 SINGLE
 this . pluginManager . onPause ( ) ;

 - SINGLE
 - this . appView . loadUrl ( STRING0 ) ;
 -
 SINGLE
 this . appView . pauseTimers ( ) ;
 }


<nb>
 COMMENT
 package NAMESPACE ;

 - import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 -
 - import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 -
 public class MainTestActivity extends BaseTestCordovaActivity {
 public static final String START_URL = STRING0 ;

 <nb> public class MainTestActivity extends BaseTestCordovaActivity {
 super . loadUrl ( url ) ;
 }

 - @ override protected void loadConfig ( ) {
 + @ override protected void loadConfig ( ) {
 super . loadConfig ( ) ;
 SINGLE
 String errorUrl = getIntent ( ) . getStringExtra ( STRING1 ) ;

Remove unused imports from MainTestActivity


<nb> public class NewRepeatTests < REMOTE_MODEL > extends DatabaseTestCase {
 }

 private void saveAndTriggerRepeatListener ( Task task ) {
 - Flags . set ( Flags . SUPPRESS_HOOKS ) ;
 + Flags . set ( Flags . ACTFM_SUPPRESS_SYNC ) ;
 if ( task . isSaved ( ) )
 taskDao . saveExisting ( task ) ;
 else
 <nb> public class RepeatTests extends DatabaseTestCase {
 }

 private void saveAndTriggerRepeatListener ( Task task ) {
 - Flags . set ( Flags . SUPPRESS_HOOKS ) ;
 + Flags . set ( Flags . ACTFM_SUPPRESS_SYNC ) ;
 if ( task . isSaved ( ) )
 taskDao . saveExisting ( task ) ;
 else


<nb> public class DiagnosticGroups {
 static final String DIAGNOSTIC_GROUP_NAMES =
 STRING0 +
 STRING1 +
 + STRING2 +
 STRING3 +
 STRING4 +
 STRING5 +
 <nb> public class DiagnosticGroups {
 PeepholeFoldConstants . SHIFT_AMOUNT_OUT_OF_BOUNDS ,
 PeepholeFoldConstants . FRACTIONAL_BITWISE_OPERAND ) ;

 + public static final DiagnosticGroup CONFORMANCE_VIOLATIONS =
 + DiagnosticGroups . registerGroup ( STRING6 ,
 + CheckConformance . CONFORMANCE_VIOLATION ,
 + CheckConformance . CONFORMANCE_POSSIBLE_VIOLATION ) ;
 +
 COMMENT
 COMMENT
 COMMENT

Add a diagnostic group for conformance tests so they can be promoted to error


<nb> import NAMESPACE ;
 public interface LivingEntity extends Entity {

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 public int getHealth ( ) ;

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public void setHealth ( int health ) ;
 +
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public int getMaxHealth ( ) ;

 COMMENT
 COMMENT


<nb> package NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class SnapshotCommand extends AbstractCommand {
 if ( schemas == null ) {
 schemas = new CatalogAndSchema [ ] { database . getDefaultSchema ( ) } ;
 }
 - DatabaseSnapshot snapshot = SnapshotGeneratorFactory . getInstance ( ) . createSnapshot ( schemas , database , snapshotControl ) ;
 +
 + ObjectQuotingStrategy originalQuotingStrategy = database . getObjectQuotingStrategy ( ) ;
 +
 + database . setObjectQuotingStrategy ( ObjectQuotingStrategy . QUOTE_ALL_OBJECTS ) ;
 + DatabaseSnapshot snapshot ;
 + try {
 + snapshot = SnapshotGeneratorFactory . getInstance ( ) . createSnapshot ( schemas , database , snapshotControl ) ;
 + } finally {
 + database . setObjectQuotingStrategy ( originalQuotingStrategy ) ;
 + }
 +
 +

 String format = getSerializerFormat ( ) ;
 if ( format == null ) {

Cannot snapshot primary keys on lower case tables


<nb> public final class ClusterServiceImpl implements ClusterService , ConnectionListe
 node . getNodeExtension ( ) . beforeJoin ( ) ;
 } catch ( Exception e ) {
 logger . warning ( e . getMessage ( ) ) ;
 - sendBeforeJoinCheckFailure ( joinRequest . getAddress ( ) , e . getMessage ( ) ) ;
 + sendBeforeJoinCheckFailure ( joinRequest . getAddress ( ) , e . getMessage ( ) ) ;
 return ;
 }
 }
 <nb> public class DefaultNodeExtension implements NodeExtension {

 @ override
 public void beforeJoin ( ) {
 - logger . info ( STRING0 ) ;
 }

 }
 <nb> public interface NodeExtension {
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 void beforeJoin ( ) ;

Check style fix and added more info to java doc


<nb> public class ScriptAPI extends ApiImplementor {

 ScriptWrapper script = new ScriptWrapper (
 params . getString ( ACTION_PARAM_SCRIPT_NAME ) ,
 - params . getString ( ACTION_PARAM_SCRIPT_DESC ) ,
 + getParam ( params , ACTION_PARAM_SCRIPT_DESC , STRING0 ) ,
 engine ,
 type ,
 true ,

Fix exception while adding script through the API


<nb> import NAMESPACE ;

 COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT


<nb> public class JarHandlerBase {

 protected static class EntryInfo {
 public EntryInfo ( final String shortName , final EntryInfo parent , final boolean directory ) {
 - this . shortName = new String ( shortName ) ;
 + this . shortName = shortName ;
 this . parent = parent ;
 isDirectory = directory ;
 }

reuse the underlying string since we store it as map key anyway


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class ExpressionInterpreter
 return node ;
 }
 else {
 - throw new RuntimeException ( STRING0 + value . getClass ( ) . getName ( ) ) ;
 + throw new UnsupportedOperationException ( STRING0 + value . getClass ( ) . getName ( ) ) ;
 }
 argumentValues . add ( value ) ;
 argumentTypes . add ( type ) ;
 <nb> public class ExpressionInterpreter
 return handle . invokeWithArguments ( argumentValues ) ;
 }
 catch ( Throwable throwable ) {
 - throw new RuntimeException ( STRING1 , throwable ) ;
 + Throwables . propagateIfInstanceOf ( throwable , RuntimeException . class ) ;
 + Throwables . propagateIfInstanceOf ( throwable , Error . class ) ;
 + throw new RuntimeException ( throwable . getMessage ( ) , throwable ) ;
 }
 }

Improve error messages from function calls


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - public class HorizontalBarChart extends ColumnChart
 + public class HorizontalBarChart extends BarChart
 {
 public HorizontalBarChart ( Context context )
 {
 <nb> package NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;

 import NAMESPACE ;


<nb> public class LoggingConfiguration {
 @ jsonproperty
 protected String logFormat ;

 - @ validationmethod ( message = STRING0 )
 + @ validationmethod ( message = STRING1 )
 public boolean isValidArchiveConfiguration ( ) {
 return ! archive || archivedLogFilenamePattern != null ;
 }


<nb> class ItemView extends LinearLayout {
 addView ( imageView , new LinearLayout . LayoutParams ( LayoutParams . WRAP_CONTENT , LayoutParams . WRAP_CONTENT ) ) ;

 final TextView textView = new TextView ( context ) ;
 + textView . setPadding ( NUMBER0 , NUMBER1 , NUMBER0 , NUMBER0 ) ;
 + textView . setTextSize ( NUMBER2 ) ;
 textView . setText ( node . displayName ( ) ) ;
 addView ( textView , new LinearLayout . LayoutParams ( LayoutParams . WRAP_CONTENT , LayoutParams . WRAP_CONTENT ) ) ;
 }


<nb> public class AttributeChecks implements ActionSplitterListener {
 return ; SINGLE
 }
 if ( isolatedRuleRef ( x . getText ( ) ) != null ) {
 - SINGLE
 - SINGLE
 + errMgr . grammarError ( ErrorType . ISOLATED_RULE_REF ,
 + g . fileName , x , x . getText ( ) , expr ) ;
 return ;
 }
 errMgr . grammarError ( ErrorType . UNKNOWN_SIMPLE_ATTRIBUTE ,


<nb> public class IcqActivator
 public void start ( BundleContext context ) throws Exception
 {
 Hashtable hashtable = new Hashtable ( ) ;
 - hashtable . put ( AccountManager . PROTOCOL_PROPERTY_NAME , STRING0 ) ;
 + hashtable . put ( AccountManager . PROTOCOL_PROPERTY_NAME , ProtocolNames . ICQ ) ;

 AccountManagerIcqImpl icqAccountManager =
 new AccountManagerIcqImpl ( ) ;

Renamed Activator java to IcqActivator java


<nb> public class ContainerUtil extends ContainerUtilRt {

 @ notnull
 @ contract ( pure = true )
 + public static < T > List < T > sorted ( @ notnull Collection < T > list , @ notnull Comparator < T > comparator ) {
 + return sorted ( ( Iterable < T > ) list , comparator ) ;
 + }
 +
 + @ notnull
 + @ contract ( pure = true )
 public static < T > List < T > sorted ( @ notnull Iterable < T > list , @ notnull Comparator < T > comparator ) {
 List < T > sorted = newArrayList ( list ) ;
 sort ( sorted , comparator ) ;


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class BinaryGrailsPlugin extends DefaultGrailsPlugin {
 }
 this . projectDirectory = isJar ? null : IOUtils . findApplicationDirectoryFile ( pluginClass ) ;

 + if ( BuildSettings . BASE_DIR != null && projectDirectory != null ) {
 + try {
 + if ( projectDirectory . getCanonicalPath ( ) . startsWith ( BuildSettings . BASE_DIR . getCanonicalPath ( ) ) ) {
 + isBase = true ;
 + }
 + } catch ( IOException e ) {
 + SINGLE
 + }
 + }
 +
 URL rootResourcesURL = IOUtils . findRootResourcesURL ( pluginClass ) ;
 if ( rootResourcesURL == null ) {
 throw new PluginException ( STRING0 + pluginClass ) ;


<nb> public class MediaRouteDescriptorAssert
 isNotNull ( ) ;
 int actualVolumeMax = actual . getVolumeMax ( ) ;
 assertThat ( actualVolumeMax ) SINGLE
 - . overridingErrorMessage ( STRING0 , volumeMax ,
 + . overridingErrorMessage ( STRING1 , volumeMax ,
 actualVolumeMax ) SINGLE
 . isEqualTo ( volumeMax ) ;
 return this ;

Correct the error message for MediaRouteDescriptorAssert hasVolumeMax int


<nb>
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 package NAMESPACE ;

 import NAMESPACE ;


<nb> public class RequestFutureTarget < R > implements FutureTarget < R > ,
 COMMENT
 @ override
 public void run ( ) {
 - request . clear ( ) ;
 + if ( request != null ) {
 + request . clear ( ) ;
 + }
 }

 COMMENT


<nb> public class TypeUtils {
 }
 }

 + SINGLE
 + for ( JetType other : resultingTypes ) {
 + if ( typeChecker . equalTypes ( other , type ) ) {
 + continue outer ;
 + }
 + }
 resultingTypes . add ( type ) ;
 }

 <nb> public class JetTypeCheckerTest extends JetLiteFixture {
 }

 public void testIntersect ( ) throws Exception {
 + assertIntersection ( STRING0 , STRING1 , STRING2 ) ;
 + assertIntersection ( STRING0 , STRING2 , STRING1 ) ;
 + assertIntersection ( STRING2 , STRING3 , STRING2 ) ;
 +
 assertIntersection ( STRING4 , STRING4 , STRING4 ) ;
 assertIntersection ( STRING5 , STRING4 , STRING5 ) ;
 assertIntersection ( STRING5 , STRING5 , STRING4 ) ;


<nb>
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT


<nb>
 package NAMESPACE ;
 + import NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 -
 - import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 -
 import NAMESPACE ;
 import NAMESPACE ;

 <nb> public class TCollectorUDPWriter extends OpenTSDBGenericWriter {
 }

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT


<nb> public class Ec2RebalanceTest extends AbstractRebalanceTest {

 @ after
 public void ec2Cleanup ( ) throws Exception {
 - stopClusterQuiet ( activeHostNames , ec2RebalanceTestConfig ) ;
 - cleanupCluster ( activeHostNames , ec2RebalanceTestConfig ) ;
 + if ( activeHostNames . size ( ) > NUMBER0 ) {
 + stopClusterQuiet ( activeHostNames , ec2RebalanceTestConfig ) ;
 + cleanupCluster ( activeHostNames , ec2RebalanceTestConfig ) ;
 + }
 }

 @ override
 <nb> public class Ec2RebalanceTest extends AbstractRebalanceTest {
 tmplNode . getPartitionIds ( ) ) ;
 nodes . add ( node ) ;
 nodeIdsInv . put ( nodeId , hostName ) ;
 + activeHostNames . add ( hostName ) ;
 }

 Cluster cluster = new Cluster ( template . getName ( ) , nodes ) ;
 - activeHostNames . addAll ( nodeIds . keySet ( ) ) ;

 deploy ( activeHostNames , ec2RebalanceTestConfig ) ;
 startClusterAsync ( activeHostNames , ec2RebalanceTestConfig , nodeIds ) ;


<nb> public class IrcStack
 public void command ( ChatRoomIrcImpl chatroom , String command )
 {
 String target ;
 - if ( command . startsWith ( STRING0 ) )
 + if ( command . toLowerCase ( ) . startsWith ( STRING0 ) )
 {
 command = command . substring ( NUMBER0 ) ;
 int endOfNick = command . indexOf ( STRING1 ) ;


<nb> public class CypherPluginFunctionalTest extends AbstractRestFunctionalTestBase {
 SINGLE
 return super . doCypherRestCall ( ENDPOINT , script , status , params ) ;
 }
 -
 -
 -
 -
 -
 -
 -
 -
 }
 <nb>
 COMMENT
 package NAMESPACE ;

 + import static NAMESPACE ;
 +
 + import NAMESPACE ;
 + import NAMESPACE ;
 +
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 - import NAMESPACE ;
 -
 - import static NAMESPACE ;
 -
 public class CypherPluginTest implements GraphHolder
 {

 <nb> public class CypherPluginTest implements GraphHolder
 @ beforeclass
 public static void startDatabase ( )
 {
 - db = new ImpermanentGraphDatabase ( STRING0
 - + System . currentTimeMillis ( ) ) ;
 -
 + db = new ImpermanentGraphDatabase ( ) ;
 }
 -
 }


<nb> public class HikariDataSource extends HikariConfig implements DataSource
 if ( username != null && ! username . equals ( otherKey . username ) ) {
 return false ;
 }
 - else if ( username != otherKey . username ) {
 + else if ( ! username . equals ( otherKey . username ) ) {
 return false ;
 }
 else if ( password != null && ! password . equals ( otherKey . password ) ) {
 return false ;
 }
 - else if ( password != otherKey . password ) {
 + else if ( ! password . equals ( otherKey . password ) ) {
 return false ;
 }


<nb>
 package NAMESPACE ;

 + SINGLE
 +
 + SINGLE
 +
 + SINGLE
 +
 + SINGLE
 +
 + SINGLE
 +
 + SINGLE
 +
 import NAMESPACE ;
 import NAMESPACE ;


<nb> import NAMESPACE ;

 import NAMESPACE ;

 - import NAMESPACE ;
 -
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 -
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;


<nb>
 package NAMESPACE ;

 public final class MocoVersion {
 - public static final String VERSION = STRING0 ;
 + public static final String VERSION ;
 +
 + static {
 + VERSION = MocoVersion . class . getPackage ( ) . getImplementationVersion ( ) ;
 + }

 private MocoVersion ( ) {
 }


<nb> public class ZWaveConfiguration implements OpenHABConfigurationService , ZWaveEve
 record . addAction ( STRING0 , STRING1 ) ;
 }
 records . add ( record ) ;
 -
 - SINGLE
 - record . addAction ( STRING2 , STRING3 ) ;
 }
 return records ;
 }
 <nb> public class ZWaveConfiguration implements OpenHABConfigurationService , ZWaveEve
 this . zController . requestRemoveFailedNode ( nodeId ) ;
 }

 - SINGLE
 - SINGLE
 - if ( action . equals ( STRING2 ) ) {
 - logger . debug ( STRING4 , nodeId ) ;
 - ZWaveVersionCommandClass versionCommandClass = ( ZWaveVersionCommandClass ) node
 - . getCommandClass ( CommandClass . VERSION ) ;
 - if ( versionCommandClass == null ) {
 - return ;
 - }
 -
 - SINGLE
 - this . zController . sendData ( versionCommandClass . getVersionMessage ( ) ) ;
 - }
 -
 SINGLE
 SINGLE
 return ;

Remove version info this is now included in the startup


<nb>
 package NAMESPACE ;

 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 public class SourceDocument extends JavaScriptObject
 <nb>
 package NAMESPACE ;

 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;


<nb> public class MavenUIUtil {

 final Color foreground = selected ? UIUtil . getTreeSelectionForeground ( ) : UIUtil . getTreeTextForeground ( ) ;

 + Color background = selected ? UIUtil . getTreeSelectionBackground ( ) : UIUtil . getTreeTextBackground ( ) ;
 +
 panel . add ( baseComponent , BorderLayout . CENTER ) ;
 - panel . setBackground ( selected ? UIUtil . getTreeSelectionBackground ( ) : UIUtil . getTreeTextBackground ( ) ) ;
 + panel . setBackground ( background ) ;
 panel . setForeground ( foreground ) ;

 CheckBoxState state = handler . getState ( userObject ) ;
 checkbox . setSelected ( state != CheckBoxState . UNCHECKED ) ;
 checkbox . setEnabled ( state != CheckBoxState . PARTIAL ) ;
 - checkbox . setBackground ( UIUtil . getTreeTextBackground ( ) ) ;
 + checkbox . setBackground ( background ) ;
 checkbox . setForeground ( foreground ) ;

 return panel ;


<nb> class ArrayKey {

 @ override
 public boolean equals ( Object obj ) {
 - if ( ! obj . getClass ( ) . isArray ( ) )
 + if ( ! ( obj instanceof ArrayKey ) )
 return false ;
 - return Arrays . equals ( a , ( Object [ ] ) obj ) ;
 + return Arrays . equals ( a , ( ( ArrayKey ) obj ) . a ) ;
 }

 @ override


<nb>
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public class DomFileDescription < T > {
 }

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

update doc for initializeFileDescription


<nb> public class ZooClient extends AbstractZooKeeperManager
 SINGLE
 SINGLE
 System . out . println ( STRING0 ) ;
 - if ( broker . getCachedMasterMachineId ( ) == getMasterNotifyId ( ) )
 + if ( broker . getMasterMachineId ( ) == getMasterNotifyId ( ) )
 {
 System . out . println ( STRING1 ) ;
 return ;
 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;

 public class ZooKeeperBroker extends AbstractBroker
 {
 <nb> public class ZooKeeperBroker extends AbstractBroker
 }
 return master . getMachineId ( ) ;
 }
 -
 - public int getCachedMasterMachineId ( )
 - {
 - return master != null ? master . getMachineId ( ) : XaLogicalLog . MASTER_ID_REPRESENTING_NO_MASTER ;
 - }

 private void connectToMaster ( Machine machine )
 {


<nb> import NAMESPACE ;
 import NAMESPACE ;

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT


<nb> import static NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 @ singleton
 @ extension ( title = STRING0 , version = STRING1 )
 <nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 @ singleton
 @ extension ( title = STRING2 , version = STRING1 )


<nb> public class AccountManager {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public void invalidateAuthToken ( final String accountType , final String authToken ) {
 if ( accountType == null ) throw new IllegalArgumentException ( STRING0 ) ;
 - if ( authToken == null ) throw new IllegalArgumentException ( STRING1 ) ;
 try {
 - mService . invalidateAuthToken ( accountType , authToken ) ;
 + if ( authToken != null ) {
 + mService . invalidateAuthToken ( accountType , authToken ) ;
 + }
 } catch ( RemoteException e ) {
 SINGLE
 throw new RuntimeException ( e ) ;

allow null authtokens to invalidateAuthToken


<nb> public class SRTMPlugin extends OsmandPlugin {
 } else {
 if ( hillshadeLayer != null ) {
 mapView . removeLayer ( hillshadeLayer ) ;
 + hillshadeLayer = null ;
 activity . refreshMap ( ) ;
 }
 }


<nb> public class DatePickerDialog extends DialogFragment implements
 Log . d ( TAG , STRING0 ) ;
 getDialog ( ) . getWindow ( ) . requestFeature ( Window . FEATURE_NO_TITLE ) ;

 - View view = inflater . inflate ( R . layout . mdtp_date_picker_dialog , null ) ;
 + View view = inflater . inflate ( R . layout . mdtp_date_picker_dialog , container ) ;

 mDayOfWeekView = ( TextView ) view . findViewById ( R . id . date_picker_header ) ;
 mMonthAndDayView = ( LinearLayout ) view . findViewById ( R . id . date_picker_month_and_day ) ;
 <nb> public class TimePickerDialog extends DialogFragment implements
 Bundle savedInstanceState ) {
 getDialog ( ) . getWindow ( ) . requestFeature ( Window . FEATURE_NO_TITLE ) ;

 - View view = inflater . inflate ( R . layout . mdtp_time_picker_dialog , null ) ;
 + View view = inflater . inflate ( R . layout . mdtp_time_picker_dialog , container ) ;
 KeyboardListener keyboardListener = new KeyboardListener ( ) ;
 view . findViewById ( R . id . time_picker_dialog ) . setOnKeyListener ( keyboardListener ) ;

Properly use the ViewGroup when inflating the picker layouts


<nb> public abstract class BaseTest {
 }

 private String locateTool ( String tool ) {
 - String [ ] roots = { STRING0 , STRING1 , STRING2 } ;
 + String [ ] roots = { STRING0 , STRING1 } ;
 for ( String root : roots ) {
 if ( new File ( root + tool ) . exists ( ) ) {
 return root + tool ;

removed reference to local node installation


<nb> public class CqlConfigHelper
 liveRemoteHosts . add ( host ) ;
 }

 - public void onSuspected ( Host host )
 - {
 - }
 -
 @ override
 public HostDistance distance ( Host host )
 {


<nb> class OptionsUsage {
 String flagName = getFlagName ( optionField ) ;
 String typeDescription = getTypeDescription ( optionField ) ;
 Option annotation = optionField . getAnnotation ( Option . class ) ;
 - usage . append ( STRING0 ) . append ( plainFlagName ) . append ( STRING1 ) ;
 + usage . append ( STRING2 ) . append ( plainFlagName ) . append ( STRING1 ) ;
 usage . append ( flagName ) . append ( STRING3 ) ;
 if ( annotation . abbrev ( ) != STRING4 ) {
 usage . append ( STRING5 ) . append ( annotation . abbrev ( ) ) . append ( STRING6 ) ;


<nb> public abstract class CloseableResource < T > implements Closeable {
 COMMENT
 COMMENT
 COMMENT
 + @ override
 public abstract void close ( ) ;
 }


<nb> public class BuildAndDecodeTest {

 @ test
 public void qualifiersTest ( ) throws BrutException {
 - compareValuesFiles ( STRING0
 - + STRING1
 - + STRING2 ) ;
 + if ( ! System . getProperty ( STRING3 ) . toLowerCase ( ) . contains ( STRING4 ) ) {
 + compareValuesFiles ( STRING0
 + + STRING1
 + + STRING2 ) ;
 + }
 }

 @ test

Windows build fix skip checking files in values for test application


<nb> public class LayoutManager extends RecyclerView . LayoutManager {
 markerLine = r . bottom ;
 }

 + if ( sd . headerParams . isHeaderSticky ( ) && r . top < NUMBER0 ) {
 + r . top = NUMBER0 ;
 + r . bottom = r . top + sd . headerHeight ;
 + }
 +
 layoutDecorated ( header , r . left , r . top , r . right , r . bottom ) ;

 return markerLine ;

Correctly layout stickied inline headers


<nb> public class FS20Binding extends AbstractActiveBinding < FS20BindingProvider >
 if ( config != null ) {

 Boolean configChanged = false ;
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 String baudRateString = ( String ) config . get ( KEY_BAUD_RATE ) ;
 if ( StringUtils . isNotBlank ( baudRateString ) ) {
 properties . put ( KEY_BAUD_RATE , baudRateString ) ;
 configChanged = true ;
 }

 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 -
 String parityString = ( String ) config . get ( KEY_PARITY ) ;
 if ( StringUtils . isNotBlank ( parityString ) ) {
 - properties . put ( KEY_PARITY , Integer . parseInt ( parityString ) ) ;
 + properties . put ( KEY_PARITY , parityString ) ;
 configChanged = true ;
 }

put a String in the map for KEY_PARITY not an Integer


<nb>
 package NAMESPACE ;

 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class JetChangeSignatureTest extends LightCodeInsightTestCase {
 JetChangeSignatureHandler . createDialog ( element , file . findElementAt ( editor . getCaretModel ( ) . getOffset ( ) ) , getProject ( ) , editor ) ;
 assertNotNull ( dialog ) ;
 dialog . canRun ( ) ;
 + Disposer . register ( getTestRootDisposable ( ) , dialog . getDisposable ( ) ) ;
 return dialog . evaluateChangeInfo ( ) ;
 }


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class GithubCreateGistAction extends DumbAwareAction {
 }

 LOG . error ( STRING0 ) ;
 - throw new InvalidStateException ( STRING0 ) ;
 + throw new IllegalStateException ( STRING0 ) ;
 }

 @ nullable


<nb> public class NetworkClient implements KafkaClient {
 try {
 this . selector . poll ( Utils . min ( timeout , metadataTimeout , requestTimeoutMs ) ) ;
 } catch ( IOException e ) {
 - log . error ( STRING0 , e ) ;
 + log . error ( STRING1 , e ) ;
 }

 SINGLE

Fix logging message in NetworkClient poll not to mention producer


<nb> public abstract class RealmObject {
 }

 void onCompleted ( Long handoverRowPointer ) {
 - realm . checkIfValid ( ) ;
 if ( ! isCompleted ) {
 isCompleted = true ;
 long nativeRowPointer = TableQuery . nativeImportHandoverRowIntoSharedGroup ( handoverRowPointer , realm . sharedGroupManager . getNativePointer ( ) ) ;
 <nb> public class RealmResults < E extends RealmObject > extends AbstractList < E > {
 COMMENT
 COMMENT
 private boolean onCompleted ( ) {
 - realm . checkIfValid ( ) ;
 try {
 long tvHandover = pendingQuery . get ( ) ; SINGLE
 SINGLE


<nb> public class EmbeddableInterfaceTest {

 @ embeddable
 public interface EmbeddableInterface {
 -
 +
 + String getName ( ) ;
 }

 @ embeddable
 public class EmbeddableClass implements EmbeddableInterface {
 +
 + @ override
 + public String getName ( ) {
 + return null ;
 + }

 }

 @ test
 - public void test ( ) {
 + public void Type ( ) {
 assertEquals (
 QEmbeddableInterfaceTest_EmbeddableInterface . class ,
 QEmbeddableInterfaceTest_EntityClass . entityClass . children . any ( ) . getClass ( ) ) ;
 }

 + @ test
 + public void Properties ( ) {
 + assertNotNull ( QEmbeddableInterfaceTest_EmbeddableInterface . embeddableInterface . name ) ;
 + assertNotNull ( QEmbeddableInterfaceTest_EmbeddableClass . embeddableClass . name ) ;
 + }
 +
 }


<nb> public class RemoteServerListConfigurable extends MasterDetailsComponent impleme
 @ override
 public void reset ( ) {
 myRoot . removeAllChildren ( ) ;
 - List < RemoteServer < ? > > servers = getServers ( ) ;
 - for ( RemoteServer < ? > server : servers ) {
 + for ( RemoteServer < ? > server : getServers ( ) ) {
 addServerNode ( server , false ) ;
 }
 super . reset ( ) ;
 }

 - private List < RemoteServer < ? > > getServers ( ) {
 + private List < ? extends RemoteServer < ? > > getServers ( ) {
 if ( myServerType == null ) {
 return myServersManager . getServers ( ) ;
 }
 else {
 - SINGLE
 - return ( List < RemoteServer < ? > > ) ( ( List ) myServersManager . getServers ( myServerType ) ) ;
 + return myServersManager . getServers ( myServerType ) ;
 }
 }


<nb> public class ByteBufferedChannelManager {
 DEFAULT_NUMBER_OF_CONNECTION_RETRIES ) ;
 this . isSpillingAllowed = configuration . getBoolean ( STRING0 , DEFAULT_ALLOW_SPILLING ) ;

 - LOG . info ( STRING1
 + LOG . info ( STRING2
 + ( this . isSpillingAllowed ? STRING3 : STRING4 ) ) ;

 SINGLE
 <nb> public class ByteBufferedChannelManager {
 }

 COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 public void shutdown ( ) {

 - LOG . info ( STRING5 ) ;
 + LOG . info ( STRING6 ) ;

 SINGLE
 this . incomingConnectionThread . interrupt ( ) ;


<nb> public class TitanRexsterApplication extends AbstractMapRexsterApplication {

 private static void configureExtensions ( final List < HierarchicalConfiguration > extensionConfigurations , final RexsterApplicationGraph rag ) {
 if ( extensionConfigurations != null ) {
 - rag . loadExtensionsConfigurations ( extensionConfigurations ) ;
 + SINGLE
 + SINGLE
 }
 }

 <nb> public class TitanRexsterApplication extends AbstractMapRexsterApplication {
 final List < String > allowableNamespaces = new ArrayList < String > ( ) { {
 add ( STRING0 ) ;
 } } ;
 -
 - rag . loadAllowableExtensions ( allowableNamespaces ) ;
 + SINGLE
 + SINGLE
 }
 }

Commented out inaccessible methods in Rexster


<nb> public class RepeatTestsGtasksSyncRemote extends RepeatTestsGtasksSync {
 completionDate = dueDate - DateUtilities . ONE_DAY ;
 else
 completionDate = dueDate + DateUtilities . ONE_DAY ;
 - < < < < < < < HEAD
 remoteModel . completed = GtasksApiUtilities . unixTimeToGtasksCompletionTime ( completionDate ) ;
 - == == == =
 - remoteModel . completed = GtasksApiUtilities . unixTimeToGtasksDate ( completionDate ) ;
 - > > > > > > > sbosley / NUMBER0 sb_repeat_tests
 remoteModel . status = STRING0 ;
 try {
 gtasksService . updateGtask ( GtasksApiUtilities . extractListIdFromSelfLink ( remoteModel ) , remoteModel ) ;


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 public interface VersionMatcher extends Comparator < String > {
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 public VersionSelector createSelector ( String selectorString ) ;
 }
 <nb> import NAMESPACE ;
 import NAMESPACE ;

 public interface VersionSelector extends Comparator < String > {
 + COMMENT
 + COMMENT
 + COMMENT
 public boolean isDynamic ( ) ;

 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 public boolean requiresMetadata ( ) ;

 + COMMENT
 + COMMENT
 + COMMENT
 public boolean matchesUniqueVersion ( ) ;

 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 public boolean accept ( String candidate ) ;

 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 public boolean accept ( ComponentMetadata candidate ) ;
 }

Added javadocs for VersionSelector and VersionMatcher


<nb> public class HomeActivity extends Activity {
 SINGLE
 SINGLE
 SINGLE
 - final int firstVisibleItemPos = layoutManager . findFirstCompletelyVisibleItemPosition ( ) ;
 - if ( firstVisibleItemPos <= NUMBER0 && toolbar . getTranslationZ ( ) != NUMBER0 ) {
 + final int firstVisibleItemPos = layoutManager . findFirstVisibleItemPosition ( ) ;
 + if ( firstVisibleItemPos == NUMBER0
 + && layoutManager . findViewByPosition ( NUMBER0 ) . getTop ( ) == grid . getPaddingTop ( )
 + && toolbar . getTranslationZ ( ) != NUMBER0 ) {
 toolbar . setTranslationZ ( FLOAT0 ) ;
 - } else if ( firstVisibleItemPos > NUMBER0 && toolbar . getTranslationZ ( ) != - FLOAT1 ) {
 + } else if ( toolbar . getTranslationZ ( ) != - FLOAT1 ) {
 toolbar . setTranslationZ ( - FLOAT1 ) ;
 }
 }


<nb> public class Modules implements Iterable < Module >

 public void registerParentsIfMissing ( BaseHome basehome , StartArgs args , Module module ) throws IOException
 {
 - for ( String name : module . getParentNames ( ) )
 + for ( String name : new ArrayList < > ( module . getParentNames ( ) ) )
 {
 if ( ! modules . containsKey ( name ) )
 {

Fixed start jar handling of expanded modules during add to start


<nb> public class RenderEntityBlock extends Render {
 lightZ = ( int ) ( Math . floor ( entity . posZ ) + kBase ) ;

 GL11 . glDisable ( NUMBER0 COMMENT ) ;
 - renderBlock ( util , world , lightX , lightY , lightZ , false , true ) ;
 + renderBlock ( util , world , NUMBER1 , NUMBER1 , NUMBER1 , lightX , lightY , lightZ , false , true ) ;
 GL11 . glEnable ( NUMBER0 COMMENT ) ;
 GL11 . glPopMatrix ( ) ;


<nb> public class RightCurlyCheck extends AbstractCheck {
 case TokenTypes . LITERAL_DO :
 nextToken = ast . findFirstToken ( TokenTypes . DO_WHILE ) ;
 lcurly = ast . findFirstToken ( TokenTypes . SLIST ) ;
 - rcurly = lcurly . getLastChild ( ) ;
 + if ( lcurly != null ) {
 + rcurly = lcurly . getLastChild ( ) ;
 + }
 break ;
 default :
 SINGLE
 <nb> public class InputRightCurlyDoWhile {
 public void foo9 ( ) {
 do { } while ( true ) ;
 }
 +
 + public void foo10 ( ) {
 + do ; while ( true ) ;
 + }
 }

fixed NPE in RightCurly when do has no curlies


<nb> public class ToggleScrollView extends ScrollView
 SINGLE
 SINGLE
 boolean result = super . onInterceptTouchEvent ( ev ) ;
 - if ( mDetector . onTouchEvent ( ev ) )
 + SINGLE
 + if ( ev . getAction ( ) == MotionEvent . ACTION_DOWN || mDetector . onTouchEvent ( ev ) )
 {
 return result ;
 }
 <nb> public class ToggleScrollView extends ScrollView
 return false ;
 }

 - SINGLE
 + SINGLE
 + SINGLE
 class YScrollDetector extends GestureDetector . SimpleOnGestureListener
 {
 @ override

Propagate the ACTION_DOWN event to the ScrollView so we can stop scrolling after a fling


<nb> public class SkeletonControl extends AbstractControl implements Cloneable {
 COMMENT
 COMMENT
 COMMENT
 - private transient boolean hwSkinningDesired = false ;
 + private transient boolean hwSkinningDesired = true ;

 COMMENT
 COMMENT


<nb> public class ImportingJob implements Jsonizable {
 JSONUtilities . safePut ( cfg , STRING0 , false ) ;
 this . config = cfg ;

 + lastTouched = System . currentTimeMillis ( ) ;
 +
 dir . mkdirs ( ) ;
 }


<nb> public interface Entity {
 COMMENT
 COMMENT
 public void setFireTicks ( int ticks ) ;
 +
 + COMMENT
 + COMMENT
 + COMMENT
 + public void remove ( ) ;
 }

Added Entity remove which marks an entity for removal


<nb> public class LocationManagerService extends ILocationManager . Stub {
 return ;
 }

 - SINGLE
 - if ( mWakeLockAcquireTime != NUMBER0 ) {
 - handleLocationChangedLocked ( location ) ;
 - }
 + handleLocationChangedLocked ( location ) ;

 if ( ( mWakeLockAcquireTime != NUMBER0 ) &&
 ( SystemClock . elapsedRealtime ( ) - mWakeLockAcquireTime

Fix timing problem that was introduced when we removed location provider polling


<nb> public class CopyPasteIndentProcessor implements CopyPastePostProcessor < IndentTr
 int lineNumber = initialDocument . getTextLength ( ) > caretOffset ? initialDocument . getLineNumber ( caretOffset )
 : initialDocument . getLineCount ( ) - NUMBER0 ;
 final int offset = getLineStartSafeOffset ( initialDocument , lineNumber ) ;
 - final int caretColumn = caretOffset - offset ;

 if ( bounds . getStartOffset ( ) == offset ) {
 String toString = initialDocument . getText ( TextRange . create ( offset , initialDocument . getLineEndOffset ( lineNumber ) ) ) ;
 <nb> public class CopyPasteIndentProcessor implements CopyPastePostProcessor < IndentTr
 return ch != STRING0 ;
 }
 } ) ;
 - if ( toIndent < NUMBER1 || toString . startsWith ( STRING1 ) ) {
 - toIndent = caretColumn ;
 + if ( ( toIndent < NUMBER1 || toString . startsWith ( STRING1 ) ) && initialText . length ( ) >= caretOffset ) {
 + toIndent = caretOffset - offset ;
 }
 }
 else if ( isNotApplicable ( initialDocument , offset ) )

use the proper caret offset in case of selection


<nb> public class CompilerWorkspaceConfiguration implements PersistentStateComponent <

 public boolean COMPILE_IN_BACKGROUND = false ;
 public boolean AUTO_SHOW_ERRORS_IN_EDITOR = true ;
 - @ deprecated public final boolean CLOSE_MESSAGE_VIEW_IF_SUCCESS = true ;
 + @ deprecated public boolean CLOSE_MESSAGE_VIEW_IF_SUCCESS = true ;
 public boolean COMPILE_DEPENDENT_FILES = false ;
 public boolean CLEAR_OUTPUT_DIRECTORY = false ;
 public boolean ASSERT_NOT_NULL = true ;

Final fields cause exception in XML serializer


<nb> public class HttpPanel extends AbstractPanel {
 this . setLayout ( new GridBagLayout ( ) ) ;
 this . setSize ( NUMBER0 , NUMBER1 ) ;
 gridBagConstraints1 . gridx = NUMBER2 ;
 - gridBagConstraints1 . gridy = NUMBER2 ;
 + gridBagConstraints1 . gridy = NUMBER3 ;
 gridBagConstraints1 . weightx = FLOAT0 ;
 gridBagConstraints1 . weighty = FLOAT0 ;
 gridBagConstraints1 . fill = java . awt . GridBagConstraints . BOTH ;
 <nb> public class HttpPanel extends AbstractPanel {
 gridBagConstraints4 . anchor = java . awt . GridBagConstraints . SOUTHWEST ;
 gridBagConstraints4 . fill = java . awt . GridBagConstraints . HORIZONTAL ;
 gridBagConstraints4 . gridx = NUMBER2 ;
 - gridBagConstraints4 . gridy = NUMBER3 ;
 + gridBagConstraints4 . gridy = NUMBER2 ;
 gridBagConstraints4 . weightx = FLOAT1 ;
 - this . add ( getSplitVert ( ) , gridBagConstraints1 ) ;
 + SINGLE
 this . add ( getJPanel ( ) , gridBagConstraints4 ) ;
 + this . add ( getSplitVert ( ) , gridBagConstraints1 ) ;
 }
 COMMENT


<nb> public class HikariConfig implements HikariConfigMBean
 LOGGER . debug ( prop + ( value != null ? value : STRING0 ) ) ;
 }
 catch ( Exception e ) {
 + continue ;
 }
 }
 }


<nb> public final class TreeUtil {

 public static List < TreePath > collectExpandedPaths ( final JTree tree , TreePath path ) {
 final ArrayList < TreePath > result = new ArrayList < TreePath > ( ) ;
 + if ( ! tree . isExpanded ( path ) ) return result ;
 final Object lastPathComponent = path . getLastPathComponent ( ) ;
 final TreeModel model = tree . getModel ( ) ;
 if ( model . isLeaf ( lastPathComponent ) ) {


<nb> public class TextEditingTarget implements
 @ handler
 void onExtractLocalVariable ( )
 {
 + if ( ! isCursorInRMode ( ) )
 + {
 + showRModeWarning ( STRING0 ) ;
 + return ;
 + }
 +
 docDisplay_ . focus ( ) ;

 String initialSelection = docDisplay_ . getSelectionValue ( ) ;
 <nb> public class TextEditingTarget implements
 ) ;
 }

 + private void showRModeWarning ( String command )
 + {
 + globalDisplay_ . showMessage ( MessageDisplay . MSG_WARNING ,
 + STRING1 ,
 + STRING2 + command + STRING3 +
 + STRING4 ) ;
 + return ;
 + }
 +
 +
 @ handler
 void onExtractFunction ( )
 {
 + if ( ! isCursorInRMode ( ) )
 + {
 + showRModeWarning ( STRING5 ) ;
 + return ;
 + }
 +
 docDisplay_ . focus ( ) ;

 String initialSelection = docDisplay_ . getSelectionValue ( ) ;

don t allow extract function and extract variable when not in R mode


<nb> public class RepackZipEntriesStepTest {
 String unzipExpected = Joiner . on ( STRING0 ) . join ( new ImmutableList . Builder < String > ( )
 . add ( STRING1 )
 . add ( STRING2 )
 - . add ( STRING3 ) . add ( dir . getPath ( ) )
 + . add ( STRING3 ) . add ( dir . getAbsolutePath ( ) )
 . add ( inApk )
 . addAll ( entries )
 . build ( ) ) ;
 <nb> public class RepackZipEntriesStepTest {
 assertEquals ( zipExpected , zipStep . getShellCommand ( context ) ) ;

 SINGLE
 - assertEquals ( zipStep . getWorkingDirectory ( ) , dir ) ;
 + assertEquals ( zipStep . getWorkingDirectory ( ) . getAbsolutePath ( ) , dir . getAbsolutePath ( ) ) ;

 verify ( context ) ;
 }

Fix path inconsistency in RepackZipEntriesStepTest


<nb> public class ListView extends AbsListView {
 }
 final Paint paint = mDividerPaint ;

 - final int listBottom = mBottom - mTop - mListPadding . bottom + mScrollY ;
 + int effectivePaddingTop = NUMBER0 ;
 + int effectivePaddingBottom = NUMBER0 ;
 + if ( ( mGroupFlags & CLIP_TO_PADDING_MASK ) == CLIP_TO_PADDING_MASK ) {
 + effectivePaddingTop = mListPadding . top ;
 + effectivePaddingBottom = mListPadding . bottom ;
 + }
 +
 + final int listBottom = mBottom - mTop - effectivePaddingBottom + mScrollY ;
 if ( ! mStackFromBottom ) {
 int bottom = NUMBER0 ;

 <nb> public class ListView extends AbsListView {
 }
 } else {
 int top ;
 - int listTop = mListPadding . top ;
 + int listTop = effectivePaddingTop ;

 final int scrollY = mScrollY ;

Fix a bug that could cause padded lists to draw dividers incorrectly


<nb> public class ShareCompat {
 COMMENT
 public String getHtmlText ( ) {
 String result = mIntent . getStringExtra ( IntentCompat . EXTRA_HTML_TEXT ) ;
 - if ( mIntent == null ) {
 + if ( result == null ) {
 CharSequence text = getText ( ) ;
 if ( text instanceof Spanned ) {
 result = Html . toHtml ( ( Spanned ) text ) ;

Fix a bug in ShareCompat IntentReader getHtmlText fallback


<nb> public class SpdyClientProvider implements ClientProvider {
 }

 private static SpdyClientConnection createSpdyChannel ( StreamConnection connection , Pool < ByteBuffer > bufferPool ) {
 - SpdyChannel spdyChannel = new SpdyChannel ( connection , bufferPool , null , new ByteBufferSlicePool ( BufferAllocator . BYTE_BUFFER_ALLOCATOR , NUMBER0 , NUMBER0 ) ) ;
 + SpdyChannel spdyChannel = new SpdyChannel ( connection , bufferPool , null , new ByteBufferSlicePool ( BufferAllocator . BYTE_BUFFER_ALLOCATOR , NUMBER1 , NUMBER1 ) ) ;
 return new SpdyClientConnection ( spdyChannel ) ;
 }


<nb> public final class HttpHelper {
 SINGLE
 String baseUrl ;
 relativeUrl = endpoint . getHttpUri ( ) . toASCIIString ( ) ;
 + SINGLE
 + if ( relativeUrl . contains ( STRING0 ) ) {
 + relativeUrl = ObjectHelper . before ( relativeUrl , STRING0 ) ;
 + }
 if ( url . startsWith ( relativeUrl ) ) {
 baseUrl = url . substring ( NUMBER0 , relativeUrl . length ( ) ) ;
 relativeUrl = url . substring ( relativeUrl . length ( ) ) ;

Fixed url rewrite issue after a recent bug fix


<nb> public abstract class PackageManager {
 public static final String FEATURE_ETHERNET = STRING0 ;

 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + @ sdkconstant ( SdkConstantType . FEATURE )
 + public static final String FEATURE_HDMI_CEC = STRING1 ;
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public final class SystemServer {

 mSystemServiceManager . startService ( MediaSessionService . class ) ;

 - mSystemServiceManager . startService ( HdmiControlService . class ) ;
 + if ( mPackageManager . hasSystemFeature ( PackageManager . FEATURE_HDMI_CEC ) ) {
 + mSystemServiceManager . startService ( HdmiControlService . class ) ;
 + }

 mSystemServiceManager . startService ( TvInputManagerService . class ) ;


<nb> abstract class AbstractCommandLineRunner < A extends Compiler ,
 STRING0 ,
 STRING1 ) ;

 + static final String WAITING_FOR_INPUT_WARNING =
 + STRING2 ;
 +
 private final CommandLineConfig config ;

 private final PrintStream defaultJsOutput ;
 <nb> abstract class AbstractCommandLineRunner < A extends Compiler ,
 throw new FlagUsageException ( STRING3 +
 STRING4 ) ;
 }
 +
 + this . err . println ( WAITING_FOR_INPUT_WARNING ) ;
 inputs . add ( SourceFile . fromInputStream ( STRING5 , System . in , inputCharset ) ) ;
 usingStdin = true ;
 }

Emit a warning message when the compiler is waiting for input


<nb> import NAMESPACE ;

 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Add a comment to the EventListenerInvoker class


<nb> public class RajawaliRenderer implements GLSurfaceView . Renderer , INode {

 public static boolean supportsUIntBuffers = false ;

 - private boolean mSceneInitialized ;
 + protected boolean mSceneInitialized ;
 COMMENT
 COMMENT
 COMMENT

Allow subclasses to fiddle with mSceneInitialized


<nb> public enum AndroidExcludedRefs {
 }
 } ,

 - TEXT_VIEW__MLAST_HOVERED_VIEW ( SAMSUNG . equals ( MANUFACTURER ) && SDK_INT == KITKAT ) {
 + TEXT_VIEW__MLAST_HOVERED_VIEW (
 + SAMSUNG . equals ( MANUFACTURER ) && SDK_INT >= KITKAT && SDK_INT <= LOLLIPOP ) {
 @ override void add ( ExcludedRefs . Builder excluded ) {
 SINGLE
 excluded . staticField ( STRING0 , STRING1 ) ;

extend TextView mLastHoveredView exclusion to Lollipop


<nb> public class ByteDecoder extends Bytes {
 _pool . done ( this ) ;
 }

 - final static SimplePool < ByteDecoder > _pool = new SimplePool < ByteDecoder > ( STRING0 , NUM_ENCODERS * NUMBER0 , - NUMBER1 ) {
 + final static SimplePool < ByteDecoder > _pool = new SimplePool < ByteDecoder > ( STRING0 , NUM_ENCODERS * NUMBER0 , NUM_ENCODERS * NUMBER2 ) {

 protected ByteDecoder createNew ( ) {
 if ( D ) System . out . println ( STRING1 ) ;
 <nb> public class ByteEncoder extends Bytes {
 _pool . done ( this ) ;
 }

 - final static SimplePool < ByteEncoder > _pool = new SimplePool < ByteEncoder > ( STRING2 , NUM_ENCODERS , - NUMBER1 ) {
 + final static SimplePool < ByteEncoder > _pool = new SimplePool < ByteEncoder > ( STRING2 , NUM_ENCODERS , NUM_ENCODERS * NUMBER3 ) {
 protected ByteEncoder createNew ( ) {
 if ( D ) System . out . println ( STRING3 ) ;
 return new ByteEncoder ( ) ;


<nb> public class Factory implements Configuration {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 @ deprecated
 public final < T > Field < T > constant ( Object value , Class < ? extends T > type ) {
 <nb> public class Factory implements Configuration {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 @ deprecated
 public final < T > Field < T > constant ( Object value , Field < T > field ) {
 <nb> public class Factory implements Configuration {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 @ deprecated
 public final < T > Field < T > constant ( Object value , DataType < T > type ) {


<nb> public class SwingBuilder extends BuilderSupport {
 factories . put ( name , factory ) ;
 }

 + public Object getConstraints ( ) {
 + return constraints ;
 + }
 +
 public LinkedList getContainingWindows ( ) {
 return containingWindows ;
 }

allow private constraints to be retrieved as a read only property


<nb> public abstract class FindManager {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public abstract class FindManager {
 }

 COMMENT
 + COMMENT
 + COMMENT
 + public abstract String getStringToReplace ( @ notnull String foundString , @ notnull FindModel model ) throws MalformedReplacementStringException ;
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 - public abstract String getStringToReplace ( @ notnull String foundString , @ notnull FindModel model ) throws MalformedReplacementStringException ;
 public abstract String getStringToReplace ( @ notnull String foundString , @ notnull FindModel model ,
 int startOffset , @ notnull CharSequence documentText ) throws MalformedReplacementStringException ;


<nb> public class WebView extends AbsoluteLayout
 SINGLE
 if ( ! rect . equals ( mLastVisibleRectSent ) ) {
 Point pos = new Point ( rect . left , rect . top ) ;
 + mWebViewCore . removeMessages ( EventHub . SET_SCROLL_OFFSET ) ;
 mWebViewCore . sendMessage ( EventHub . SET_SCROLL_OFFSET ,
 nativeMoveGeneration ( ) , mUserScroll ? NUMBER0 : NUMBER1 , pos ) ;
 mLastVisibleRectSent = rect ;

Prevent SET_SCROLL_OFFSET spam from clogging tubes


<nb> public abstract class InputRegistry {
 InputState inputState = getRunningInputState ( input . getId ( ) ) ;

 if ( inputState != null ) {
 - input . stop ( ) ;
 + try {
 + input . stop ( ) ;
 + } catch ( Exception e ) {
 + LOG . warn ( STRING0 , input . getId ( ) , e ) ;
 + }
 removeFromRunning ( input ) ;
 inputState . setState ( InputState . InputStateType . STOPPED ) ;
 finishedStop ( inputState ) ;

Fixing handling of failed attempts to stop input


<nb> public class CsrfPreventionFilter extends FilterBase {

 HttpSession session = req . getSession ( false ) ;

 - @ suppresswarnings ( STRING0 )
 LruCache < String > nonceCache = ( session == null ) ? null
 : ( LruCache < String > ) session . getAttribute (
 Constants . CSRF_NONCE_SESSION_ATTR_NAME ) ;


<nb> public class AutoValueProcessor extends AbstractProcessor {
 STRING0 ,
 STRING1 ,
 STRING2 +
 - STRING3 ,
 + STRING4 ,
 STRING5 ,
 STRING6 ,
 SINGLE

Fix somewhat less than shocking mistake in generation of toString


<nb> public final class MetricName {
 private int hash = NUMBER0 ;

 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public final class MetricName {

 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT