Jeongseok (JS) Lee

I added a checklist to the description accordingly. Feel free to modify if necessary.

There is a WIP branch for this: [`feature/ompl_max_extent`](https://github.com/personalrobotics/aikido/tree/feature/ompl_max_extent). It would be nice if someone takes over it.

This suggestion sounds good to me. Could we have this in [`STYLE.md`](https://github.com/personalrobotics/aikido/blob/master/STYLE.md) as well?

@psigen At first we also intended to apply DRY whenever applicable, but we encountered a rather tricky case. Let me take a simple example: ```cpp class A { public: void...

It's a compile-time error due to `A::foo()` is hidden (see [this live code](https://godbolt.org/g/VgKDW1)), and unfortunately, the code still doesn't work for the same reason. > Would this mean that every...

> Thus it might be addressed by a good FAQ entry describing the issue and a note in the STYLE.md. :+1: > Separately, any thoughts on whether simultaneous virtual and...

According to [this comment](https://github.com/lvandeve/lodepng/pull/39#issuecomment-238247702) of the Author, this library is meant to be used as the source code itself rather than binaries. :smile:

It could be. @lvandeve may have the right answer for you. I just replied as this reminds me the comment. :sweat:

FYI, cleaner diff: https://github.com/lvandeve/lodepng/pull/60/files?w=1

You could find one [here](https://github.com/flexible-collision-library/fcl/blob/master/include/fcl/narrowphase/detail/convexity_based_algorithm/gjk_libccd-inl.h#L1467).