fmin/fmax not defined

Bug #383247 reported by reed
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Won't Fix
Undecided
Unassigned

Bug Description

src/live_effects/lpe-dynastroke.cpp uses fmin and fmax.

Some platforms don't have that. So I added to
src/live_effects/lpe-dynastroke.cpp :

#define fmin(a,b) ((a)<(b)?(a):(b))
#define fmax(a,b) ((a)>(b)?(a):(b))

Also work/inkscape-21256/src/display/nr-filter.cpp refers to fmax but it
is commented out.

Revision history for this message
reed (reed-reedmedia) wrote :

This problem continues in snapshot 22440.

tags: added: build
su_v (suv-lp)
tags: added: build-compile-code-design
jazzynico (jazzynico)
tags: added: code-design
removed: build-compile-code-design
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

This is no longer necessary in C++11 as the functions became part of the standard in <cmath>

Changed in inkscape:
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.