-Match vs -Eq #14

Open
opened 2018-11-27 15:16:28 -05:00 by ichorflowsfreely · 0 comments
ichorflowsfreely commented 2018-11-27 15:16:28 -05:00 (Migrated from github.com)

4e2d22a71d/Task/Topological-sort/PowerShell/topological-sort.psh (L32)

There are several lines from line 32 down that use -match instead of using -eq. This I believe is in error due to match being a substring match [while it might work for their test set it incorrectly identifies matches when its a substring]. I didn't want to update in rosetta code without having someone verify and yours seems to be the most complete code backup of it, so if this is the wrong place, i'm sorry.

https://github.com/acmeism/RosettaCodeData/blob/4e2d22a71d0542105c66e329e5fe295b0905b0d7/Task/Topological-sort/PowerShell/topological-sort.psh#L32 There are several lines from line 32 down that use -match instead of using -eq. This I believe is in error due to match being a substring match [while it might work for their test set it incorrectly identifies matches when its a substring]. I didn't want to update in rosetta code without having someone verify and yours seems to be the most complete code backup of it, so if this is the wrong place, i'm sorry.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: aparler/RosettaCodeData#14
No description provided.