BNUACM / bnuoj-vjudge

Virtual Judge for BNUOJ
GNU Affero General Public License v3.0
13 stars 7 forks source link

fix ural vj status parsing #6

Closed crccw closed 9 years ago

crccw commented 9 years ago

The author filter is no longer supported by the status page, so we have get runid in the response header and use that to locate the right row. Currently it will fetch only 10 runs initailly, if our runid cannot be found then the number of runs fetched will increase in 10 until timeout.

I also update the regex and language table, the latter fix the java problem

Review on Reviewable

51isoft commented 9 years ago

Comments from the review on Reviewable.io


vjudge-v2/UralJudger.cpp, line 104 [r1] (raw file): Don't create new Bott here. Or you have to delete this pointer before every throw.


vjudge-v2/UralJudger.cpp, line 111 [r1] (raw file): Let's just set it after.


vjudge-v2/UralJudger.cpp, line 131 [r1] (raw file): If it reaches maximum, also throw exception.


crccw commented 9 years ago

Comments from the review on Reviewable.io


vjudge-v2/UralJudger.cpp, line 104 [r1] (raw file): Done.


vjudge-v2/UralJudger.cpp, line 111 [r1] (raw file): Done.


vjudge-v2/UralJudger.cpp, line 131 [r1] (raw file): Done.


51isoft commented 9 years ago

Comments from the review on Reviewable.io


vjudge-v2/UralJudger.cpp, line 102 [r2] (raw file): nit: count = 10


vjudge-v2/UralJudger.cpp, line 129 [r2] (raw file): nit: (count == 100) {


crccw commented 9 years ago

Comments from the review on Reviewable.io


vjudge-v2/UralJudger.cpp, line 102 [r2] (raw file): Done.


vjudge-v2/UralJudger.cpp, line 129 [r2] (raw file): Done.