Pradeep Krishnamurthy

Results 77 comments of Pradeep Krishnamurthy

what is the fix here? is it about adding a QL_REQUIRE to check the value? Thanks.

Hello, I would like to give this a go, if no one else is working on it. It's a lot of new concepts for me, I will be happy to...

Now that the minimal initial is done, I would like to continue working on the next step. As I said before, a lot of new concepts for me here, apologies...

Is `640` the expected ciphertext length for sha1? If yes, I would love to know why.

Now I am ready for hopefully the last part of this issue. This might be both interesting and challenging. This is what I gather after snooping around in the code...

I agree, the previous PR got overly lengthy, not proud of it. This is a completely new and unrelated field for me, which could be one reason. I will dig...

Hello, Is this issue still open? I am happy to give it a go.

Hello, is this issue still open? I am new to the project, happy to work on this with some guidance :)

> The logic can be improved by saying that a single space must not separate 2 unquoted string fields. By that logic, wouldn't the following with a single space between...

Yeah, I agree, so the logic you proposed `single space must not separate 2 unquoted string fields` may solve `badges.data` but it will incorrectly parse other datasets like the example...