Ticket #22 (closed patch: fixed)

Opened 2 years ago

Last modified 1 year ago

Replace C style casts in byte order functions

Reported by: cneumann Assigned to: cneumann
Priority: trivial Milestone:
Component: System Version: 2.0
Keywords: Cc:
Completion:

Description

The byte order functions osghtonXX and osgntohXX have a bunch of C style casts in them, and I don't like those ;)

Attachments

BaseFunctions_ByteOrder.diff (26.3 kB) - added by cneumann on 09/23/06 15:58:04.
patch

Change History

(follow-up: ↓ 2 ) 09/04/06 17:32:54 changed by dirk

I don't like those in the current form anyway, as there is no way to explicitly convert between big and little endian. If we touch them we might fix that one, too.

09/23/06 15:58:04 changed by cneumann

  • attachment BaseFunctions_ByteOrder.diff added.

patch

(in reply to: ↑ 1 ) 09/23/06 16:01:59 changed by cneumann

  • status changed from new to assigned.
  • type changed from task to patch.

Replying to dirk:

I don't like those in the current form anyway, as there is no way to explicitly convert between big and little endian. If we touch them we might fix that one, too.

Dirk, could you take a look at the patch I attached and tell me if this fits with what you had in mind, please ?

10/14/06 05:58:48 changed by cneumann

  • status changed from assigned to closed.
  • resolution set to fixed.

I committed this as r286.