old c-style functions in common.cpp

Bug #846069 reported by Aron Boyette
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
New Real Time Battle
Won't Fix
Wishlist
Unassigned

Bug Description

~/common/common_rl/common.cpp contains various c-style functions. Among them are...
changecase
upcase
downcase
gsub
split
trim
The components that call these functions should be modified to call the corresponding methods in the latest string class (which I think is part of STL, but I'm not sure). Afterwards, these functions should perhaps be removed from common.cpp altogether.

Tags: beta cleanup
Rick Stovall (fpstovall)
Changed in nrtb:
status: New → Triaged
importance: Low → Wishlist
tags: added: beta cleanup
Revision history for this message
Rick Stovall (fpstovall) wrote :

The nrtb::tcp_socket and the nrtb::conf_reader are the primary users of the functions listed above.

Oh and for the record these are NOT "C style" functions.. they are in modern in both design and operation. They are type-safe, throw if they have a problem, do not return status flags and use modern syntax and the STL. They are free-standing functions, but that's the only only similarity to classic "C" functions; they would not even compile in C.

So there! :)

Rick Stovall (fpstovall)
Changed in nrtb:
status: Triaged → 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.