Use generics for DatasetProducer.hasExpired() map parameter

  • Timothy Anyona

    Timothy Anyona - 2013-07-24

    Just a request to have this modification in the next release.

  • Ulf Dittmer

    Ulf Dittmer - 2013-07-25

    I'm trying to remember why I genericized the produceDataset method in the first place. Changing the signature in an interface for little gain doesn't seem to make a lot of sense, as it forces folks to update their code. I really can't remember; it would seem to have been a not-so-good decision.

    Changing the ChartPostProcessor interface was rather different, that still makes sense.

  • Timothy Anyona

    Timothy Anyona - 2013-07-25

    Fair enough. I'll add @SuppressWarnings.

  • Ulf Dittmer

    Ulf Dittmer - 2013-07-25

    If folks (you or someone else) thinks this is of substantial value, I'll certainly consider it, but right now I'm still stumped why I changed the other method signature. Genericizing is good, but breaking an interface for what now looks like little value - not so much.

  • Timothy Anyona

    Timothy Anyona - 2013-07-26

    I don't think it's that big of a deal. Only brought it up after noticing the change to the produceDataset method, then I wondered why not this other one.


Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.

No, thanks