reta commented on PR #1033:
URL: https://github.com/apache/cxf/pull/1033#issuecomment-1326921258
> Below is another example of misbehavior because `REMOVED_MARKER` is passed
outside of MessageContentsList class:
>
Yes, the choice of `REMOVED_MARKER` is unclear to me
> I might not completely understand this. Do you mean for existing code, or
when developing new one assuming that parameter can be omitted?
I mean the fix should treat (imho) these two examples properly: when
`projectId` is not present, it has to apply the correct placeholder instead of
`REMOVED_MARKER` (assuming this flow has to be supported):
```
public boolean allocate(Integer projectId, Object targetIds, Integer[]
parameterIds) { }
public boolean allocate(int projectId, Object targetIds, Integer[]
parameterIds) { }
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
