[ 
https://issues.apache.org/jira/browse/MINIFICPP-910?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16858498#comment-16858498
 ] 

Arpad Boda edited comment on MINIFICPP-910 at 6/7/19 10:54 AM:
---------------------------------------------------------------

Thinking a bit more while having my coffee, I found that the risk of this 
change is pretty low. 

I didn't change the implementation of the existing functions, just moved them 
from a class to a namespace. Which naturally changes linkage, so we should 
verify on target platforms, but the behaviour doesn't change, so in case it 
compiles it should be ok. 


was (Author: aboda):
Thinking a bit more while having my coffee, I found that the risk of this 
change is pretty low. 

I didn't change the implementation of the existing function, just moved them 
from a class to a namespace. Which naturally changes linkage, so we should 
verify on target platforms, but the behaviour doesn't change, so in case it 
compiles it should be ok. 

> Extend StringUtils with string join capability
> ----------------------------------------------
>
>                 Key: MINIFICPP-910
>                 URL: https://issues.apache.org/jira/browse/MINIFICPP-910
>             Project: Apache NiFi MiNiFi C++
>          Issue Type: Improvement
>            Reporter: Arpad Boda
>            Assignee: Arpad Boda
>            Priority: Major
>             Fix For: 0.7.0
>
>          Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> String join is frequently needed and MiNiFi code contains a lot of 
> copy-pasted implementations for that. A general solution should be provided. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to