units conversion (was: about contibuting to octave)

Shai Ayal shaiay at gmail.com
Mon Apr 20 05:27:52 CDT 2009


On Mon, Apr 20, 2009 at 12:17 PM, xianghang liu <xianghangliu at gmail.com> wrote:
> I tried again and found fltk_backend.oct could be built successfully with a
> few modifications of fltk_backend.cc. The modification I made was moving
> "#ifdef min ... #endif" which is originally in the middle of "#include ..."
> instructions to the end them. Now it compiles fine and works well.

I see this change was introduced by Michael in changeset 7965:
http://hg.savannah.gnu.org/hgweb/octave/rev/6a6a030a3517
with the comment "Fix compilation under MSVC".
Will moving the #ifdef block below all includes break MSVC?
If not then I say move it.

Shai


More information about the Octave-maintainers mailing list