fritzo / jenn3d

A Toy for Visualizing Coxeter Polytopes
jenn3d.org
Other
41 stars 8 forks source link

Patch 1 #1

Closed asimba closed 11 years ago

fritzo commented 11 years ago

Hi Alexey, this looks great. It's so nice to have another set of eyes on the code. Could you remove those now-obsolete color comments, and then I'll merge?

asimba commented 11 years ago

Hi Fritz, Ok. Comments are removed. :) Four eyes see more than two. For now I can't find reason of the infinite loop at void Drawing::display_vertex (int v) (only in Windows, and only at hide restore window). I have to read about glut event loops... :)

2013/6/21 Fritz Obermeyer notifications@github.com

Hi Alexey, this looks great. It's so nice to have another set of eyes on the code. Could you remove those now-obsolete color comments, and then I'll merge?

— Reply to this email directly or view it on GitHubhttps://github.com/fritzo/jenn3d/pull/1#issuecomment-19828500 .

asimba commented 11 years ago

The rhetorical question: where is strict ANSI/POSIX compatibility in Windows (MSDN said - it's there,but...)

2013/6/21 Alexey Simbarsky asimbarsky@gmail.com

Hi Fritz, Ok. Comments are removed. :) Four eyes see more than two. For now I can't find reason of the infinite loop at void Drawing::display_vertex (int v) (only in Windows, and only at hide restore window). I have to read about glut event loops... :)

2013/6/21 Fritz Obermeyer notifications@github.com

Hi Alexey, this looks great. It's so nice to have another set of eyes on the code. Could you remove those now-obsolete color comments, and then I'll merge?

— Reply to this email directly or view it on GitHubhttps://github.com/fritzo/jenn3d/pull/1#issuecomment-19828500 .