Opened on 05/02/2016 at 01:09:46 PM

#4011 new change

The types of character and used in the code are inconsistent.

Reported by: sergz Assignee:
Priority: P5 Milestone:
Module: Adblock-Plus-for-Internet-Explorer Keywords: goodfirstbug
Cc: oleksandr, eric@… Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

Take a look for example at src/plugin/PluginFilter.cpp
firstTag is wchar_t
filterSuffix is std::wstring
'*', '[', '.', '#' are char.
In addition, std::iswalnum should be used instead of isalnum.

It is correct for the currently used characters but the code looks dirty.

What to change

  • check places and files
  • use L'<character>' and wide versions of functions.

Attachments (0)

Change History (0)

Add Comment

Modify Ticket

Change Properties
Action
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from (none).
Next status will be 'reviewing'.
 
Note: See TracTickets for help on using tickets.