[TS-1764] Unify definitions of MAX() and MIN()

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.3.2
• Component/s: Core
• Assignee: Leif Hedstrom
• Reporter: Leif Hedstrom
• Created: 20/Mar/13 16:52
• Updated: 01/Apr/13 19:43
• Resolved: 01/Apr/13 19:43

Description

There are a few duplications of these definitions, this would be splendid to unify into one definition for each (yes amc, I know there's std::max .

loki (10:48) 892/0 $ tsg 'MAX\(' | grep define
/home/leif/apache/trafficserver.git/proxy/PluginVC.cc: #define MAX(x,y) (((x) >= (y)) ? (x) : (y))
/home/leif/apache/trafficserver.git/proxy/MuxVC.cc: #define MAX(x,y) (x >= y) ? x : y;
/home/leif/apache/trafficserver.git/iocore/net/P_InkBulkIO.h: #define MAX(x, y) (x > y ? x : y)
loki (10:48) 893/0 $ tsg 'MIN\(' | grep define
/home/leif/apache/trafficserver.git/proxy/http/HttpTunnel.cc: #define MIN(x,y) ((x) <= (y)) ? (x) : (y);
/home/leif/apache/trafficserver.git/proxy/PluginVC.cc: #define MIN(x,y) (((x) <= (y)) ? (x) : (y))
/home/leif/apache/trafficserver.git/proxy/InkAPITestTool.cc: #define MIN(_x, _y) ((_x < _y) ? _x : _y)
/home/leif/apache/trafficserver.git/proxy/MuxVC.cc: #define MIN(x,y) (x <= y) ? x : y;
/home/leif/apache/trafficserver.git/example/thread-pool/psi.c: #define MIN(x,y) ((x < y) ? x :y)
/home/leif/apache/trafficserver.git/iocore/net/NetVCTest.cc: #define MIN(x,y) (x <= y) ? x : y;