Valerie (Yu-Ching) Peng wrote:

Looks fine except the following minor nit:
- Why not return realm on line 1224 after a match is found? Doesn't seem necessary to continue w/ the whole for-loop.
Indeed, looks like it is missing a "break".

-Alan.

Reply via email to