From cd11bc699ac50af4f560ed5f2e5e7903de0898b8 Mon Sep 17 00:00:00 2001 From: Arnaud Versini Date: Fri, 9 May 2014 10:38:04 +0200 Subject: C string usage improvment Change-Id: I5c59f0d2d1b911ffa1ee251e0f1355d137616493 Signed-off-by: Stephan Bergmann --- sal/qa/osl/process/osl_Thread.cxx | 5 +---- sal/qa/rtl/ostring/rtl_str.cxx | 14 +++++--------- 2 files changed, 6 insertions(+), 13 deletions(-) (limited to 'sal/qa') diff --git a/sal/qa/osl/process/osl_Thread.cxx b/sal/qa/osl/process/osl_Thread.cxx index f5f43140e2eb..1122c2518c02 100644 --- a/sal/qa/osl/process/osl_Thread.cxx +++ b/sal/qa/osl/process/osl_Thread.cxx @@ -1919,10 +1919,7 @@ namespace osl_ThreadData // at first, set the data a value char* pc = new char[2]; char m_nData = 'm'; -// LLA: this is a copy functions only and really only for \0 terminated strings -// m_nData is not a string, it's a character -// strcpy(pc, &m_nData); - memcpy(pc, &m_nData, 1); + pc[0] = m_nData; pc[1] = '\0'; myThreadData.setData(pc); diff --git a/sal/qa/rtl/ostring/rtl_str.cxx b/sal/qa/rtl/ostring/rtl_str.cxx index 116191d353c4..759d30e9836a 100644 --- a/sal/qa/rtl/ostring/rtl_str.cxx +++ b/sal/qa/rtl/ostring/rtl_str.cxx @@ -711,10 +711,9 @@ namespace rtl_str void trim_WithLength_001() { char const *pStr = " trim this"; - sal_Char *pStr2 = (sal_Char*)malloc(strlen(pStr) + 1); + sal_Char *pStr2 = (sal_Char*)strdup(pStr); if (pStr2) { - strcpy(pStr2, pStr); rtl_str_trim_WithLength( pStr2, 2 ); CPPUNIT_ASSERT_MESSAGE("string should be empty", strlen(pStr2) == 0); @@ -725,10 +724,9 @@ namespace rtl_str void trim_WithLength_002() { char const *pStr = "trim this"; - sal_Char *pStr2 = (sal_Char*)malloc(strlen(pStr) + 1); + sal_Char *pStr2 = (sal_Char*)strdup(pStr); if (pStr2) { - strcpy(pStr2, pStr); rtl_str_trim_WithLength( pStr2, 5 ); CPPUNIT_ASSERT_MESSAGE("string should contain 'trim'", strlen(pStr2) == 4); @@ -739,7 +737,7 @@ namespace rtl_str void trim_WithLength_003() { char const *pStr = " trim this"; - sal_Char *pStr2 = (sal_Char*)malloc(strlen(pStr) + 1); + sal_Char *pStr2 = (sal_Char*)strdup(pStr); if (pStr2) { strcpy(pStr2, pStr); @@ -753,10 +751,9 @@ namespace rtl_str void trim_WithLength_004() { char const *pStr = "\r\n\t \n\r trim \n this"; - sal_Char *pStr2 = (sal_Char*)malloc(strlen(pStr) + 1); + sal_Char *pStr2 = (sal_Char*)strdup(pStr); if (pStr2) { - strcpy(pStr2, pStr); rtl_str_trim_WithLength( pStr2, 17 ); CPPUNIT_ASSERT_MESSAGE("string should contain 'trim'", strlen(pStr2) == 4); @@ -767,10 +764,9 @@ namespace rtl_str void trim_WithLength_005() { char const *pStr = "\r\n\t \n\r trim \t this \n\r\t\t "; - sal_Char *pStr2 = (sal_Char*)malloc(strlen(pStr) + 1); + sal_Char *pStr2 = (sal_Char*)strdup(pStr); if (pStr2) { - strcpy(pStr2, pStr); rtl_str_trim_WithLength( pStr2, strlen(pStr2) ); CPPUNIT_ASSERT_MESSAGE("string should contain 'trim'", strlen(pStr2) == 11); -- cgit