Skip to content

Minimize reflection into Configuration #3127

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 27, 2020
Merged

Minimize reflection into Configuration #3127

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4151dbf
Minimize reflection into Configuration
idodeclare Sep 11, 2019
754ebdf
Address review feedback
idodeclare Apr 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion dev/checkstyle/suppressions.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ Portions Copyright (c) 2018-2020, Chris Fraire <[email protected]>.
|Summary\.java|OGKUnifiedHighlighter\.java|ResponseHeaderFilter\.java|BoundedBlockingObjectPool\.java|
|ObjectPool\.java|ObjectValidator\.java|ObjectFactory\.java|AbstractObjectPool\.java|
|BlockingObjectPool\.java|OGKTextVecField\.java|OGKTextField\.java|
|LazilyInstantiate\.java|PortChecker\.java" />
|LazilyInstantiate\.java|PortChecker\.java|CloseableReentrantReadWriteLock\.java|
|ResourceLock\.java" />

<suppress checks="ParameterNumber" files="CtagsReader\.java|Definitions\.java|
|JFlexXrefUtils\.java|FileAnalyzerFactory\.java|SearchController\.java|
Expand Down
33 changes: 33 additions & 0 deletions ...-indexer/src/main/java/org/opengrok/indexer/configuration/ConfigurationValueConsumer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
/*
* CDDL HEADER START
*
* The contents of this file are subject to the terms of the
* Common Development and Distribution License (the "License").
* You may not use this file except in compliance with the License.
*
* See LICENSE.txt included in this distribution for the specific
* language governing permissions and limitations under the License.
*
* When distributing Covered Code, include this CDDL HEADER in each
* file and include the License file at LICENSE.txt.
* If applicable, add the following below this CDDL HEADER, with the
* fields enclosed by brackets "[]" replaced with your own identifying
* information: Portions Copyright [yyyy] [name of copyright owner]
*
* CDDL HEADER END
*/

/*
* Copyright (c) 2020, Chris Fraire <[email protected]>.
*/

package org.opengrok.indexer.configuration;

/**
* Represents an API for consuming a {@link Configuration} instance along with
* a specified value.
*/
@FunctionalInterface
public interface ConfigurationValueConsumer<V> {
void accept(Configuration configuration, V v);
}
Loading