Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#479 closed defect (fixed)

Omega's freemem.cc should use safeunistd.h, not unistd.h

Reported by: Charlie Hull Owned by: Olly Betts
Priority: normal Milestone: 1.0.21
Component: Omega Version: 1.2.0
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Operating System: Microsoft Windows

Description

....otherwise it won't build on Windows. The patch should be obvious in this case

Change History (3)

comment:1 by Olly Betts, 14 years ago

Milestone: 1.0.21
Resolution: fixed
Status: newclosed

Fixed in trunk r14527, and backported to 1.0 in r14528.

It would be useful to actually implement get_free_physical_memory() using the win32 api though...

comment:2 by Charlie Hull, 14 years ago

Looking at freemem.cc in the current trunk, there appears to be a Win32 implementation already.

comment:3 by Olly Betts, 14 years ago

Yes, there was an implementation already in the copy of this code in xapain-core's perftest, so I copied that over.

Note: See TracTickets for help on using tickets.