[CLOSED] Improve console output of LiveDevelopment errors (#1200)

[core-ai-bot]

Issue by DennisKehrig
Wednesday Jul 11, 2012 at 17:41 GMT
Originally opened as adobe/brackets#1232

---

Before:
Some arguments of method 'Debugger.setBreakpointByUrl' can't be processed

After:
Some arguments of method 'Debugger.setBreakpointByUrl' can't be processed
Parameter 'lineNumber' has wrong type. It must be 'Number'.
Parameter 'url' has wrong type. It must be 'String'.
[expandable error object]

---

DennisKehrig included the following code: https://github.com/adobe/brackets/pull/1232/commits

[core-ai-bot]

Comment by DennisKehrig
Wednesday Jul 11, 2012 at 17:43 GMT

---

See #1200

[core-ai-bot]

Comment by gruehle
Wednesday Jul 11, 2012 at 21:49 GMT

---

Thanks! Merging.

[core-ai-bot]

Comment by peterflynn
Wednesday Jul 11, 2012 at 23:44 GMT

---

Oops, it looks like this violates #1096 though. @DennisKehrig, are you planning any more work in this file? If so, we could wait and roll up the fix into a later pull request, instead of making a whole commit just for this one-line change...

[core-ai-bot]

Comment by DennisKehrig
Thursday Jul 12, 2012 at 03:22 GMT

---

@peterflynn No, not right now... I just went ahead and added another pull request.

I'm surprised this isn't mentioned in the Coding Conventions. Is it mentioned somewhere else?
I noticed some discussion about $.each vs. Array.forEach, too - ideally I'd like to find information on what to use for iterating objects vs. arrays in a central place.

[core-ai-bot]

Comment by DennisKehrig
Thursday Jul 12, 2012 at 08:24 GMT

---

Thanks, guys!