Ben Grabham

Results 83 comments of Ben Grabham

I'm not sure whether that is overloading `@jsonize` too much though. Is there a way to pass it in as a parameter to the mixin? That seems like the best...

Try doing it like this: ``` d int fieldsFound = 0; foreach(field; objectFields) { if(field in json) { this.field = json[field]; ++fieldsFound; } else if(!field.isOptional()) { throw new JsonParsingException(getExtraAndMissingFields(json)); }...

We probably want the default to be strict. The reason for this is, if someone forgets to not ignore extra keys, no error message will be shown. It will not...

Yes, I tend to split it up into two parts: - **User is interfacing with an external API** (my current use case): The user usually writes a wrapper which is...

This is amazing, thank you @Paulche. It would be great if it either makes it into the default homebrew repo (as @monouser7dig suggested) or if @mattrajca hosts the homebrew tap...

Other problem is that this repo is dead... 👎

@brachi-wernick This is already done for `ArgumentCaptor`: https://github.com/mockito/mockito/blob/41c5606349e13a1e3e060ab8348b412662219d62/src/main/java/org/mockito/ArgumentCaptor.java#L148-L150 It doesn't make it completely typed as you can accidentally give it `Object` or some inherited class but it is a nice...

@thandy1212 I'm just stating what already exists in Mockito, not something new. I assume it was done because you don't really care about what subclass it is implementing. If you...

I agree with @sivaraam but would go one step further and make the extra visitor generic: ``` public T accept(Generic*Visitor visitor); ``` This would: - Allow CopyVisitor to be written....