# Compiling warnings should be fixed

- Thread: https://externals.io/message/55267
- Original archive: https://news-web.php.net/php.internals/55267
- Started by Laruence on 2011-09-07T01:15:32+00:00
- Messages: 3
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #55267 — Laruence — 2011-09-07T01:15:32+00:00

URL: https://externals.io/message/55267#55267

Hi:
    there is a lot of compiling warnings in truk(branch also),  I have
fixed most of them unless those in Zend/*
    here is the warning log: http://pastebin.com/ah3qtP6A (compile with

   I think it is better to fix them all, since we should release a
perfect project(although most of them are no issue)

   of course I can do it, but  need a zend karma.

 thanks

-- 
Laruence  Xinchen Hui
http://www.laruence.com/

---

## #55273 — Pierre Joye — 2011-09-07T06:26:03+00:00

URL: https://externals.io/message/55267#55273 · In reply to: #55267

Hi Laruence,

I fully agree, the amount of warnings is way too big and only seeing
the delta between two builds is useful right now.

Btw, if you have patches for Zend, please send them over so we can
review and test them :)

Cheers,


> Hi:
>    there is a lot of compiling warnings in truk(branch also),  I have
> fixed most of them unless those in Zend/*
>    here is the warning log: http://pastebin.com/ah3qtP6A (compile with
>
>   I think it is better to fix them all, since we should release a
> perfect project(although most of them are no issue)
>
>   of course I can do it, but  need a zend karma.
>
>  thanks
>
> --
> Laruence  Xinchen Hui
> http://www.laruence.com/
>



-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org

---

## #55276 — Laruence — 2011-09-07T08:25:51+00:00

URL: https://externals.io/message/55267#55276 · In reply to: #55273

Hi:
  sure , most of them is const <-> no-const,  also some incompatible
pointer type.

  I have submitted a patch for trunk,  https://bugs.php.net/bug.php?id=55629

  if there is no problem, I will make a patch for 5.4 branch too.

  in further,  and after I set up a VS environ, I will try to fix the
warnings in VS too.

thanks


2011/9/7 Pierre Joye <pierre.php@gmail.com>:
> Hi Laruence,
>
> I fully agree, the amount of warnings is way too big and only seeing
> the delta between two builds is useful right now.
>
> Btw, if you have patches for Zend, please send them over so we can
> review and test them :)
>
> Cheers,
>
> 
> > […]
>
>
>
> --
> Pierre
>
> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org
>



-- 
Laruence  Xinchen Hui
http://www.laruence.com/