The FCKDebug code should not be included in the compressed files
We don't need anything from FCKDebug when normally using the editor. We should be able so to avoid including its code inside the compressed files to save some bytes from it. The solution would be loading fckdebug.js by request, only if debugging is enabled.
Our code may have some debugging calls laying around, so the compressed files should still contain the debugging functions, but those should be empty. We could use @Packager.Remove.Start and @Packager.Remove.End to easily achieve that.
Change History (7)
| Keywords: |
Review? added
|
| Owner: |
set to Martin Kou
|
| Status: |
new →
assigned
|
| Keywords: |
Review- added; Review? removed
|
| Owner: |
changed from Martin Kou to Frederico Caldeira Knabben
|
| Status: |
assigned →
new
|
| Keywords: |
Review? added; Review- removed
|
| Status: |
new →
assigned
|
| Keywords: |
Review+ added; Review? removed
|
| Resolution: |
→ fixed
|
| Status: |
assigned →
closed
|
I had a different idea actually. I still wanted to make it possible to debug with the distributed code, even when running the compressed scripts. The proposed patch instead completely remove the debug code from the distribution, and also don't include it at all in the compressed files, which may case runtime bugs if any FCKDebug call remained in the code.
All this is quite difficult to explain in words, so I'll just attach a new patch for it.