[Asterisk-code-review] issue ASTERISK-26282 (asterisk[master])

chris de rock asteriskteam at digium.com
Thu Aug 25 00:20:48 CDT 2016


chris de rock has posted comments on this change.

Change subject: issue ASTERISK-26282
......................................................................


Patch Set 1:

to Line 166: i followed the instructions on https://wiki.asterisk.org/wiki/display/AST/Coding+Guidelines and used 'indent -i4 -ts4 -br -brs -cdw -lp -ce -nbfda -npcs -nprs -npsl -nbbo -saf -sai -saw -cs -l90 foo.c' as described. if the code has now a wrong format, the instructions may be wrong.

to Line 249: so, just add the const

to Line 350: yepp, its the old code. but i let the code in to show how the fix works. its commented out and dont affect the function.

-- 
To view, visit https://gerrit.asterisk.org/3578
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I939aa2a694148cc1054dd75ec0c47c47f47c90fb
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: chris de rock <chris at derock.de>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: chris de rock <chris at derock.de>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list