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

Bob Fels commented on CELIX-410:
--------------------------------

Quick response! Thx for the fix. I've reviewed your code updates (including 
test update). If the test sais OK, I say OK too :)

> Property loader duplicates spaces and tabs
> ------------------------------------------
>
>                 Key: CELIX-410
>                 URL: https://issues.apache.org/jira/browse/CELIX-410
>             Project: Celix
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 1.0.0-incubating
>            Reporter: Bob Fels
>
> In  framework / private / src / properties.c in function static void 
> parseLine(const char* line, properties_pt props):
> When a space or tab is parsed within a property, its outputted double, this 
> happens because the following 2 blocks of code both output this space/tab:
>  235                         else {
>  236                                 output[outputPos++] = line[linePos];
>  237                                 updateBuffers(&key, &value, &output, 
> outputPos, &key_len, &value_len);
>  238                         }
> and 
>  284                 else { //normal character
>  285                         precedingCharIsBackslash = false;
>  286                         output[outputPos++] = line[linePos];
>  287                         updateBuffers(&key, &value, &output, outputPos, 
> &key_len, &value_len);
>  288                 }
> I think lines 235..238 are unintended, but feel free to fix it any way you 
> like of course.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to