Ted Lyngmo

Results 20 comments of Ted Lyngmo

I see it's failing in Ubuntu and CentOS, presumably because of the skipped tests? Any idea what could cause it to skip the tests? ``` #86 Does not return a...

> > Any idea what could cause it to skip the tests? > > You might already have found the reason, but the command `DEBUG SLEEP` which is needed by...

> I'm hesitant to merge changes to the `sds` library because this code is also in [Redis itself](https://github.com/redis/redis/blob/ff3a3577f29a07e965b3b38d07894410d28da499/src/sds.h#L88) so we try to minimise divergence. I totally understand that. If I...

I'm not sure I understand, but is this a bug? If not, questions regarding how to use `yaml-cpp` should be asked on SO: "_If you have questions about how to...

Btw, I think you'll get the expected result if you start the function with `YAML::Node findNode = YAML::Clone(m_node);`

I see what you mean. I made this small example to show the diff betweeing `Clone()`ing vs. using the copy constructor. ``` #include #include #include #include #include #include #include #include...

Ah... sorry for the code spam. I see you already put up an example, but perhaps the above shows it even more clearly.

I don't know `CMake` very well, but instead of turning `-Weffc++` off completely for those compilers, would it be possible to make it disable the warning only for the line...

I agree that adding the compiler specific pragma directly into the code isn't nice. I was thining of tagging it somehow so that `CMake` could turn it into whatever the...

"_this option is a known bug_" - In order to document it, can you link to the bug in bugzilla?