search

zen0wu / topcoder-greed

greedy editor for topcoder arena
Apache License 2.0
229 stars 45 forks source link

Add .ValueListLength field to ParamValue. #49

Closed vexorian closed 10 years ago

vexorian commented 10 years ago

I was trying the tmpl-seq branch and was about to have success creating an input file template. But to include array test arguments in ACM style I think it is useful to have a way to fetch the length of the array from the template. Tested merging this on a tmpl-seq branch clone.

zen0wu commented 10 years ago

I think this one should be merged into tmpl-seq instead of master, am I right?

vexorian commented 10 years ago

The changes are useful for both branches. Should I make a PR for tmpl-seq?

vexorian commented 10 years ago

I don't think tmpl-seq needs this change. Unless you want to include an input template in tmpl-seq.

zen0wu commented 10 years ago

Of course we need the input template in the tmpl-seq branch. I don't think we need any changes to master from now on. All changes will be merged back to master when the new features are ready.