Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 9927157: Fix include guards (Closed)

Created:
March 27, 2013, 2:58 p.m. by Felix Dahlke
Modified:
Nov. 12, 2013, 10:08 a.m.
Reviewers:
Oleksandr
Visibility:
Public.

Description

Just added the missing include guard for FilterEngine and perfixed the one for AdblockPlus.h with ADBLOCKPLUS_. All the other ones area already prefixed with ADBLOCKPLUS_.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -2 lines) Patch
M include/AdblockPlus.h View 1 chunk +2 lines, -2 lines 0 comments Download
M include/AdblockPlus/FilterEngine.h View 2 chunks +5 lines, -0 lines 0 comments Download

Messages

Total messages: 3
Felix Dahlke
March 27, 2013, 3 p.m. (2013-03-27 15:00:13 UTC) #1
Felix Dahlke
Can you have a look at this Oleksandr? It's pretty short.
April 9, 2013, 6:49 a.m. (2013-04-09 06:49:03 UTC) #2
Oleksandr
April 9, 2013, 6:51 a.m. (2013-04-09 06:51:49 UTC) #3
On 2013/04/09 06:49:03, Felix H. Dahlke wrote:
> Can you have a look at this Oleksandr? It's pretty short.

LGTM

Powered by Google App Engine
This is Rietveld