On Sun, 29 Jan 2012, Michael Friedrich wrote:

> mh, embedded perl is complicated, creates a lot of unused variable
> warnings which aren't really the case. add a patch, does not hurt to see
> that as unused variable warning.

    I just created "Feature #2292" on the "issues" board and
attached five small patches to correct the problems I encountered.

> newer compilers don't moan about that by default. normally such
> declarations shouldn't happen inside the code but on function entry
> level - by coding style policy.
> so patch is welcome.

    Included in the case above.

> if you have many patches bundled, create an issue in the core section,
> and attach all patches to it.
> we'll reference to that on the git commits then after reviewing the
> patches sent.

    I re-did all the patches to what I grabbed using "git" and used
git to format the patches that I uploaded.  Here's hoping that I
didn't trip over the tools too badly.

    As an aside, thanks for removing the PHP dependency for the base
and CGI bits -- that helped enormously and was especially welcome as
PHP wasn't really doing much useful anyway.

    Cheers!

+------------------------------------------------+---------------------+
| Carl Richard Friend (UNIX Sysadmin)            | West Boylston       |
| Minicomputer Collector / Enthusiast            | Massachusetts, USA  |
| mailto:crfri...@rcn.com                        +---------------------+
| http://users.rcn.com/crfriend/museum           | ICBM: 42:22N 71:47W |
+------------------------------------------------+---------------------+

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
icinga-users mailing list
icinga-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/icinga-users

Reply via email to