@FFDCIgnore({ NoSuchElementException.class })
    protected Map<String, String> getMpConfig(MpConfigProxyService service, ClassLoader cl, String propertyName,  Map<String, String> map) {
        try {
            String value = service.getConfigValue(cl, propertyName, String.class).trim();
            if (!value.isEmpty()) {
                map.put(propertyName, value);
            } else {
                if (TraceComponent.isAnyTracingEnabled() && tc.isDebugEnabled()) {
                    Tr.debug(tc, propertyName + " is empty. Ignore it.");
                }
            }
        } catch (NoSuchElementException e) {
            if (TraceComponent.isAnyTracingEnabled() && tc.isDebugEnabled()) {
                Tr.debug(tc, propertyName + " is not in mpConfig.");
            }
        }
        return map;
    }
@ FFDCIgnore ( { NoSuchElementException . class } ) protected Map < String , String > getMpConfig ( MpConfigProxyService service , ClassLoader cl , String propertyName , Map < String , String > map ) { try { String value = service . getConfigValue ( cl , propertyName , String . class ) . trim ( ) ; if ( ! value . isEmpty ( ) ) { map . put ( propertyName , value ) ; } else { if ( TraceComponent . isAnyTracingEnabled ( ) && tc . isDebugEnabled ( ) ) { Tr . debug ( tc , propertyName + " is empty. Ignore it." ) ; } } } catch ( NoSuchElementException e ) { if ( TraceComponent . isAnyTracingEnabled ( ) && tc . isDebugEnabled ( ) ) { Tr . debug ( tc , propertyName + " is not in mpConfig." ) ; } } return map ; }
no null check other than cl. make sure that the caller sets non null objects.