/irc-logs / mozilla / #accessibility / 2012-06-18 / end
Options:
- # Session Start: Mon Jun 18 00:00:01 2012
- # Session Ident: #accessibility
- # [00:00] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
- # [00:23] * Joins: davidb (davidb@moz-68BF56A6.dsl.bell.ca)
- # [00:23] * ChanServ sets mode: +qo davidb davidb
- # [00:25] * Quits: @davidb (davidb@moz-68BF56A6.dsl.bell.ca) (Quit: blast off!)
- # [00:31] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
- # [00:35] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Ping timeout)
- # [00:35] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Ping timeout)
- # [00:35] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [00:35] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Ping timeout)
- # [00:35] * Joins: capella_ (chatzilla@moz-DD0C7E4F.twcny.res.rr.com)
- # [00:35] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)
- # [00:35] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [00:35] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
- # [00:35] * Quits: capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com) (Ping timeout)
- # [00:36] * capella_ is now known as capella
- # [00:36] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
- # [00:36] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Ping timeout)
- # [00:37] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
- # [00:48] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
- # [00:53] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
- # [00:54] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
- # [00:54] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
- # [02:58] * Quits: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
- # [02:59] * Joins: shorlander_ (shorlander@moz-853043D6.dhcp.insightbb.com)
- # [03:02] * khuey is now known as khuey|away
- # [03:03] * khuey|away is now known as khuey
- # [03:31] * Joins: mike5w3c (MikeS@moz-C3998BBE.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [04:31] * Quits: mike5w3c (MikeS@moz-C3998BBE.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
- # [04:31] * Joins: mike5w3c (MikeS@moz-736E4479.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [04:43] * khuey is now known as khuey|away
- # [04:43] * khuey|away is now known as khuey
- # [05:27] * Quits: mike5w3c (MikeS@moz-736E4479.xgsspn.imtp.tachikawa.spmode.ne.jp) (Quit: mike5w3c)
- # [07:00] * khuey is now known as khuey|away
- # [07:13] * Joins: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [07:16] * Quits: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp) (No route to host)
- # [07:16] * Joins: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [07:18] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
- # [07:26] * Joins: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se)
- # [07:53] * Joins: surkov (surkov@539620DD.7F110E62.4C549139.IP)
- # [07:53] * ChanServ sets mode: +o surkov
- # [07:54] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
- # [08:14] * Quits: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
- # [08:16] * Quits: @surkov (surkov@539620DD.7F110E62.4C549139.IP) (Quit: surkov)
- # [08:25] * Joins: mike5w3c (MikeS@moz-6876B8D5.u05.hotspot.ne.jp)
- # [09:23] * Joins: MarcoZ (Daily@moz-7A1C740A.dip.t-dialin.net)
- # [09:23] * ChanServ sets mode: +o MarcoZ
- # [09:28] * Quits: mike5w3c (MikeS@moz-6876B8D5.u05.hotspot.ne.jp) (Quit: mike5w3c)
- # [10:15] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
- # [12:10] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
- # [12:11] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Quit: nhirata)
- # [12:16] <capella> firebot: black coffee
- # [12:16] <@firebot> capella: eww, i prefer milk with my coffee
- # [12:27] <capella> firebot: milk is cow juice
- # [12:27] <@firebot> capella: ok
- # [12:38] <@firebot> trev.saunders@gmail.com granted review for attachment 633867 on bug 757503.
- # [12:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
- # [12:41] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
- # [12:47] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Ping timeout)
- # [12:51] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
- # [12:52] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Client exited)
- # [12:53] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # [12:53] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
- # [13:35] * Quits: victorporof (victorporo@2B22C4F3.20718EE9.79933D60.IP) (Quit: Linkinus - http://linkinus.com)
- # [13:35] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
- # [13:39] * Joins: mike5w3c (MikeS@moz-D1E89A92.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [13:50] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # [14:01] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
- # [14:06] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # [14:14] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
- # [14:15] <@firebot> vinceyang15@gmail.com requested review from trev.saunders@gmail .com for attachment 633994 on bug 757503.
- # [14:15] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
- # [14:30] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [14:31] * shorlander_ is now known as shorlander
- # [14:45] * khuey|away is now known as khuey
- # [15:16] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
- # [15:16] * ChanServ sets mode: +qo davidb davidb
- # [15:16] <@MarcoZ> WordPress and BuddyPress CSS confuse me. Oh, wait, CSS in general confuses me!
- # [15:16] <icaaq> MarcoZ: lol
- # [15:28] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
- # [15:29] * Joins: victorporof (victorporo@C7255039.602BD6C.79933D60.IP)
- # [15:29] * Joins: mike5w3c_ (MikeS@moz-B94F10FE.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [15:29] * Quits: mike5w3c (MikeS@moz-D1E89A92.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
- # [15:29] * mike5w3c_ is now known as mike5w3c
- # [15:31] <@firebot> trev.saunders@gmail.com granted review for attachment 633994 on bug 757503.
- # [15:31] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
- # [15:31] <@davidb> heyo
- # [15:33] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [15:36] <@davidb> any surkov sightings?
- # [15:36] <capella> in and out all weekend
- # [15:36] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
- # [15:36] <tbsaunde> phi davidb
- # [15:37] <@davidb> heyo
- # [15:37] <@davidb> capella: thanks
- # [15:37] <capella> davidb: is ok
- # [15:38] <capella> davidb: modifed bug 570322 which you previously approved ... wanna re-approve?
- # [15:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, ---, markcapella, ASSI, random fail of image map (area) accessible creation
- # [15:39] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
- # [15:40] <@davidb> i'd want to see a patch
- # [15:43] <@firebot> markcapella@twcny.rr.com requested review from dbolter@mozilla.com for attachment 634008 on bug 570322.
- # [15:43] <@MarcoZ> capella: How did the try run on that go?
- # [15:43] <capella> Great! various android issues as usual ... image problems all corrected
- # [15:44] <@MarcoZ> capella: Nice!
- # [15:44] <@MarcoZ> icaaq: Yeah my biggest problem at the moment are color codes. There are so many ways to code these in CSS that it's really confusing.
- # [15:45] * @MarcoZ would need a tool where he could input a color code and it would give me a plain clear name such as "black", "a whiter shade of pale" or so.
- # [15:45] <icaaq> MarcoZ: then less-variables would make it easier for you :)
- # [15:46] <@MarcoZ> Yeah.
- # [15:48] <icaaq> I just started a new project where we use LESS, it's a bit difficult to know all the optimizations one can do. but I'm getting there. :)
- # [15:48] * icaaq giggles every time he looks at home.less
- # [15:51] <@MarcoZ> LOL
- # [15:51] <@firebot> dbolter@mozilla.com granted review for attachment 634008 on bug 570322.
- # [15:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, ---, markcapella, ASSI, random fail of image map (area) accessible creation
- # [15:53] <capella> davidb: I ran tests on local windows ... but on the first TRY push they showed up as errors requiring adjustments
- # [15:53] <@davidb> interesting
- # [15:54] <capella> yah - i though it was localized to linux builds but even WIN broke on the TRY
- # [15:54] * khuey is now known as khuey|away
- # [15:55] <capella> this ones nice and clean across the board
- # [15:56] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
- # [15:58] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
- # [16:00] <@MarcoZ> davidb: Was comment #63 the reason you cc'd to bug 689623? :D
- # [16:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=689623 nor, --, ---, tnikkel, ASSI, layout needs to provide information on which images are visible or likely to be visible
- # [16:00] <@firebot> vinceyang15@gmail.com requested review from trev.saunders@gmail .com for attachment 634010 on bug 757503.
- # [16:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
- # [16:02] <capella> tbsaunde: is bug 765371 still on your radar?
- # [16:02] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765371 nor, --, ---, markcapella, ASSI, decomtaminate GetSelected-RowCount / ColumnCount / CellCount on accessible tables
- # [16:05] * clown is now known as clown_mtg
- # [16:05] <@davidb> MarcoZ: heh
- # [16:05] <tbsaunde> capella: commenting now ctually
- # [16:06] <capella> thanks!
- # [16:19] * Quits: icaaq (Adium@moz-CF0EF6A6.cust.bredbandsbolaget.se) (Quit: Leaving.)
- # [16:31] * Joins: icaaq (Adium@moz-E7557AE7.telia.com)
- # [16:38] * Joins: richardschwerdtfeger (richardsch@A08B5B24.9DD4DBBF.6A7A197.IP)
- # [16:48] * @MarcoZ is playing around with the Firefox dev tools.
- # [16:55] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Client exited)
- # [16:56] <@firebot> markcapella@twcny.rr.com changed the Target Milestone on bug 570322 from --- to mozilla16.
- # [16:56] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, mozilla16, markcapella, ASSI, random fail of image map (area) accessible creation
- # [17:00] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
- # [17:03] * khuey|away is now known as khuey
- # [17:04] <@MarcoZ> davidb: With a lot of NVDA object navigation, the fields are actually useable. But I think we could do more clever things with properly labelling the different text fields, so that tabbing becomes smoother.
- # [17:05] <@davidb> MarcoZ: rypple
- # [17:05] <@davidb> ?
- # [17:05] <@MarcoZ> davidb: Sorry, "inspect element" in Firefox.
- # [17:05] <@davidb> ohhh
- # [17:05] <@davidb> MarcoZ: file patches :)
- # [17:10] <@firebot> trev.saunders@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 634010 on bug 757503.
- # [17:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
- # [17:11] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
- # [17:11] <@MarcoZ> davidb: Other tools like the style editor completely illude me what their usability concept is concerned.
- # [17:11] <@MarcoZ> They basically didn't implement *any* of the stuff that was so nicely accessible in Firebug.
- # [17:12] * clown_mtg is now known as clown
- # [17:12] <@davidb> what do you get?
- # [17:14] <@MarcoZ> In the Style editor, when I open it first, i get something that has a long data:// URL as its accessible name, and no interaction whatsoever. I can then click on something like a list with NVDA's mouse emulation, but arrowing in there doesn't five me any speech, either.
- # [17:15] <@MarcoZ> I can then tab, get some different stylesheet names, as "heading1" and "link", but no idea what these are or what to do with them.
- # [17:15] <@MarcoZ> davidb: I PM'ed ya.
- # [17:21] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
- # [17:27] * Joins: drexler (chatzilla@moz-6F22CAC7.e7jvsoe-gw.cust.sover.net)
- # [17:41] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
- # [17:46] * Quits: icaaq (Adium@moz-E7557AE7.telia.com) (Quit: Leaving.)
- # [17:55] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [17:57] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
- # [17:58] * Joins: icaaq (Adium@moz-CF0EF6A6.cust.bredbandsbolaget.se)
- # [18:01] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
- # [18:01] * ChanServ sets mode: +o firebot
- # [18:02] * Joins: davidb|mobile (yaaic@31ABB91E.F7EBA23.C3C3A9D0.IP)
- # [18:06] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
- # [18:06] * ChanServ sets mode: +o jprmc
- # [18:12] * Joins: aaronlev (aaronlev@moz-654DF715.bstnma.fios.verizon.net)
- # [18:18] <icaaq> have any of you looked at https://marketplace.mozilla.org/ yet
- # [18:24] <@eeejay> https://wiki.mozilla.org/Accessibility/Mobile/GestureInterface
- # [18:25] <aaronlev> !seen marcoz
- # [18:25] <@firebot> marcoz was last seen 70 minutes and 23 seconds ago, saying 'davidb: I PM'ed ya.' in #accessibility.
- # [18:26] <aaronlev> MarcoZ: are you gone for the day?
- # [18:26] <aaronlev> i'm looking for a blind user who knows a lot of different screen readers on Windows
- # [18:27] * Quits: davidb|mobile (yaaic@31ABB91E.F7EBA23.C3C3A9D0.IP) (Ping timeout)
- # [18:33] <@MarcoZ> aaronlev: Was in a meeting with eeejay. What's the question?
- # [18:34] <@MarcoZ> icaaq: Yes I have, and I even filed a bug.
- # [18:37] * khuey is now known as khuey|away
- # [18:37] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # [18:37] <aaronlev> MarcoZ: I PM'd you
- # [18:37] * Parts: aaronlev (aaronlev@moz-654DF715.bstnma.fios.verizon.net)
- # [18:40] <@davidb> tbsaunde: jlin is going to ask you about laptops
- # [18:42] <icaaq> MarcoZ: I just got access today, *looks not so nice* imho
- # [18:44] <tbsaunde> davidb: ok
- # [18:45] <@firebot> dbolter@mozilla.com granted review for attachment 633623 on bug 764203.
- # [18:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=764203 nor, --, ---, nobody, NEW, Don't notify pivot change if it has not actually changed
- # [18:47] <@davidb> tbsaunde: just waiting to see if the x230 is available first
- # [18:48] <@MarcoZ> icaaq: When I looked, it actually wasn't that bad with a screen reader. You'd have to look yourself or ask someone else sighted for stuff like keyboard focus visibility, though.
- # [18:49] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
- # [18:50] <tbsaunde> davidb: ok, no hurry should be around all day
- # [18:50] <@davidb> ack
- # [18:54] <icaaq> MarcoZ: I think it has some a weird structure, for example the more-link is before the content, the navigation button is not connected to the navigation items (a search form between) and stuff like that.
- # [18:54] <icaaq> MarcoZ: -some^
- # [18:55] <@MarcoZ> icaaq: I may have to revisit. When I looked a few weeks ago, it looked pretty straight-forward in NVDA, but this may have changed.
- # [18:56] <icaaq> MarcoZ: yes, I'll be filing in some bugs tonight :)
- # [19:00] <@MarcoZ> CC me on them please icaaq!
- # [19:00] * @MarcoZ is going off for the night. See you all tomorrow!
- # [19:00] <icaaq> MarcoZ: Will do :)
- # [19:02] * Quits: @MarcoZ (Daily@moz-7A1C740A.dip.t-dialin.net) (Quit: MarcoZ)
- # [19:24] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Computer has gone to sleep.)
- # [19:29] * khuey|away is now known as khuey
- # [19:30] * Joins: hub (hub@A5087023.2354C43D.D8E68FF6.IP)
- # [19:30] * ChanServ sets mode: +o hub
- # [19:32] * Joins: hub_ (hub@A5087023.2354C43D.D8E68FF6.IP)
- # [19:39] <@firebot> hub@mozilla.com requested review from dbolter@mozilla.com for attachment 634093 on bug 755048.
- # [19:40] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
- # [19:45] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
- # [19:45] * ChanServ sets mode: +o askalski
- # [19:45] <@askalski> hi everybody!
- # [19:46] <@askalski> davidb, hi!
- # [19:46] <@davidb> heyo!
- # [19:46] * @davidb starts vidyo 1:1
- # [19:46] * davidb is now known as davidb|mtg
- # [19:50] * Joins: tofumatt (tofumatt@moz-9995AABD.hosts.kicksass.ca)
- # [19:54] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
- # [19:58] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Textual IRC Client: http://www.textualapp.com/)
- # [20:02] * Joins: Mic (Instantbir@moz-108E19BC.superkabel.de)
- # [20:04] <@firebot> dbolter@mozilla.com granted review for attachment 633682 on bug 756296.
- # [20:04] <@firebot> dbolter@mozilla.com requested feedback from marco.zehe@googlemail.com for attachment 633682 on bug 756296.
- # [20:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=756296 nor, --, mozilla16, eitan, NEW, nsIAccessiblePivot move to coordinate method
- # [20:16] <@firebot> dbolter@mozilla.com granted review for attachment 634093 on bug 755048.
- # [20:16] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
- # [20:23] <@firebot> markcapella@twcny.rr.com requested review from trev.saunders@gmail .com for attachment 634116 on bug 765512.
- # [20:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765512 nor, --, ---, markcapella, ASSI, decomtaminate GetSelected (Cell / Column / Row) Indices() on accessible tables
- # [20:28] * davidb|mtg is now known as davidb
- # [20:34] * Quits: Mic (Instantbir@moz-108E19BC.superkabel.de) (Quit: Instantbird 1.2a1pre -- http://www.instantbird.com)
- # [20:38] <@hub> tbsaunde: if you prefer I can change the "default" to be an explicit fall thru for the enums
- # [20:40] <tbsaunde> hub: I'm worried that might be even more confusing when things don't work
- # [20:40] <tbsaunde> since sometimes we'd complain and other times not
- # [20:41] <@hub> tbsaunde: the thing is today you have a compilation warning and nothing else. and it has been ignored.
- # [20:42] <@hub> tbsaunde: default suppress it but suppress the safeguard for new enum values
- # [20:42] <@hub> tbsaunde: on the other hand I can explicitely handle the enums doing nothing. no warning but new enum value will cause one
- # [20:43] * Joins: arky (arky@moz-7B0110AD.mv.mozilla.com)
- # [20:43] <tbsaunde> hub: so, if it was me I'd be really tempted to check the spec to see how the compiler is allowed to screw me if I just ignore the warning
- # [20:44] <tbsaunde> the thing is I'm not entirely convinced that we shouldn't handle more values of the enum
- # [20:45] <tbsaunde> but that's hard to figure out I'd have to go dive through layout a bit, doesn't seem to bother anyone, so it seems less important than other things I could spend time on
- # [20:46] * tbsaunde shrugs
- # [20:47] <@hub> tbsaunde: the compiler will do nothing. if the case isn't handled it just does not do anything
- # [20:47] <@hub> tbsaunde: like an empty default:
- # [20:47] * Quits: drexler (chatzilla@moz-6F22CAC7.e7jvsoe-gw.cust.sover.net) (Ping timeout)
- # [20:48] <@hub> tbsaunde: the warning is here to tell you " you might have missed some cases"
- # [20:48] <@hub> now it does not seem to be doing any harm the way it is
- # [20:49] <@hub> but too much warnings kill the warnings
- # [20:49] <@hub> warnings are usefull
- # [20:49] <@davidb> i very much agree
- # [20:49] <@davidb> in gcc this is just the warning -Wswitch does
- # [20:50] <@davidb> which is turned on by -Wall
- # [20:50] <@davidb> adding the default case tells the compiler "yeah i'm not worried about missing cases here"
- # [20:50] * @davidb moves on
- # [20:53] <tbsaunde> hub: sure, but if you don't read the warnings all the time its not a big deal to have some pointless ones
- # [20:53] <@hub> I read warning ALL THE TIME
- # [20:53] <@hub> ok, lunch time
- # [20:54] <tbsaunde> davidb: I know what the warning is I'm just on the fence if I think supressing it is a great idea
- # [20:54] <tbsaunde> hub: well, then don't ;) find something better to do than watch output scroll by
- # [20:55] <@davidb> tbsaunde: so you think the warning is something we should be reminded about because it points out something to be concerned about?
- # [20:56] <@davidb> that would be a good reason
- # [20:56] <@davidb> hub: so handling the cases explicitly might be good… as per tbsaunde
- # [20:57] <@davidb> then when we add to the enum… the warning can remind us in case there is a useful reason to switch on it
- # [20:57] <@davidb> either way, I vote to get rid of warnings.
- # [20:58] <@davidb> then warnings will mean something again.
- # [20:58] <@davidb> IMO
- # [20:59] <tbsaunde> davidb: imo they already mean something
- # [20:59] <@davidb> not to everyone
- # [21:00] <tbsaunde> but basically since last summer I've had more important things to with my time than look at them
- # [21:00] <@davidb> not while they are just noise for some people
- # [21:00] <@davidb> that's fair… so let's solve it quick and move on
- # [21:00] <tbsaunde> sounds like a personal problem ;p
- # [21:00] <@davidb> heh
- # [21:00] <tbsaunde> I think if we're trying to fix something we should get it right
- # [21:01] <@davidb> yes.
- # [21:01] <tbsaunde> and be sure it is right
- # [21:01] <@davidb> i'm fine with that
- # [21:01] <tbsaunde> so, I guess another way to put it is I'd rather have warnings than paper over things
- # [21:01] <@davidb> there is a cost, you'll admit right?
- # [21:02] <tbsaunde> and being sure something is correct is neither easy or fast
- # [21:02] <@davidb> it can be.
- # [21:02] <@davidb> if trivial
- # [21:02] <tbsaunde> a cost to what warnings? sure but pretty small
- # [21:02] <@davidb> i'm not sure it is as small as you think
- # [21:02] <tbsaunde> why?
- # [21:03] <@davidb> humans.
- # [21:03] <tbsaunde> hu???
- # [21:03] <@davidb> ask ms2ger and jdm
- # [21:03] <tbsaunde> I still don't understand your point
- # [21:04] <@davidb> there are people who want to compile the codebase and see no warnings.
- # [21:04] <@davidb> blocking them has a cost.
- # [21:04] <tbsaunde> nd they're wrong
- # [21:04] <@davidb> i understand your position, i'm talking about cost.
- # [21:05] <tbsaunde> making people not introduce warnings also has a cost
- # [21:05] <tbsaunde> and making people review warning fixes also has a cost
- # [21:05] <@davidb> yes indeed.
- # [21:06] <@davidb> so
- # [21:06] <@davidb> i think this boils down to different opinions on what each cost is.
- # [21:06] <@davidb> but admitting them all is worthwhile i think.
- # [21:08] <tbsaunde> so, I admitt people who believe in fixing every warning exist
- # [21:08] <tbsaunde> but I think they're wasting their time by watching for warnings all the time and wasting my time looking at their patches
- # [21:09] <@davidb> I guess it boils down to wanting to be able to notice warnings that are worth noticing.
- # [21:09] <@davidb> Does that make sense?
- # [21:10] <capella> fwiw - i think we over do warning around here ... "hey this might happen someoday" is not useful
- # [21:10] <capella> if 99.99% it never will
- # [21:10] <@davidb> capella: are you talking about compiler warnings?
- # [21:10] <tbsaunde> I suppose
- # [21:11] <capella> yeh - abd situationbal also
- # [21:11] <capella> (spelling)
- # [21:11] <@davidb> I dunno… the compiler warning thing isn't black and white for me.
- # [21:12] <tbsaunde> davidb: I guess where I differ is I don't think its that bad to have extra warnings if you just go through and read all of them say every couple months
- # [21:12] <@davidb> or before channel migration i guess
- # [21:13] <tbsaunde> I guess that would be fine
- # [21:13] <@davidb> I'm in danger of bringing up our #pragma discussion again… but let's not go there.
- # [21:13] <tbsaunde> though maybe a bit frequent for my taste, and if anything actually is bad I'd think taking it on aurora would be fine
- # [21:13] <@davidb> sure… but with awareness that we looked at it.
- # [21:14] <@davidb> that it was noticed.
- # [21:14] <@davidb> i must admit my first paid C++ work (in 1995) required 0 errors, 0 warnings for commits
- # [21:15] <capella> ack
- # [21:15] <@davidb> it has pros and cons
- # [21:15] <@davidb> the warnings were useful to me back then… at least some percentage of the time
- # [21:16] <capella> sometimes achieving results requires warnings to be accepted as suitable level of risk
- # [21:16] <tbsaunde> they are certainly sometimes useful
- # [21:16] <capella> (forgive if i intrude :) )
- # [21:16] <@davidb> heh the key is "to me" … because i am otherwise lazy
- # [21:16] <@davidb> capella: not at all
- # [21:16] <tbsaunde> the thing is its impossible to be completely correct in all cases everywhere
- # [21:17] <@davidb> i think i understand tbsaunde's position better now
- # [21:17] <@davidb> this sort of reminds me of the oranges debate… about turning off those tests vs leaving them
- # [21:18] * Parts: tofumatt (tofumatt@moz-9995AABD.hosts.kicksass.ca)
- # [21:18] <@davidb> and the idea to come up with 'known oranges' suppression etc
- # [21:18] <@davidb> very similar actually
- # [21:18] <capella> my techie side wants to have code as pure as possible ... business side requires results .... my libra tilts
- # [21:18] <@davidb> you should see a doctor about that
- # [21:18] <capella> :p
- # [21:19] <tbsaunde> its basically the same issue
- # [21:20] <capella> right ... if theres time to do it right, thats the answer
- # [21:20] <tbsaunde> accept tests failing is a little more scary than warnings since warnings mean rough "this might not be compiled the way you want which might lead to wrong results"
- # [21:21] <@davidb> tbsaunde: and we went with leaving the oranges going… but in this case they have to be noticed/starred… which is a difference.
- # [21:21] <@davidb> yeah
- # [21:21] <tbsaunde> and tests failing are "you program appears to not working as specified"
- # [21:21] <@davidb> true
- # [21:22] <capella> could be "have you designed your test properly"?
- # [21:22] <capella> lol - devils advocate
- # [21:22] <tbsaunde> true, that's certainly more effort than checking warnings every know and again, but simple test fails aren't to bad to star imo
- # [21:22] <@davidb> capella: it could be fragile test yeah
- # [21:22] <@davidb> tbsaunde: especially when we have philor heheh
- # [21:23] <tbsaunde> true
- # [21:23] <capella> philor will mock you!
- # [21:23] <@davidb> that's how he shows his love
- # [21:23] <tbsaunde> sure, but he'll get more anoyed by having to dig through logs to find out why android isn't working
- # [21:24] <capella> he told me that once :) if you get that wrong, I will mock you mercilessly
- # [21:24] <capella> android needs lots of love
- # [21:28] <@davidb> ok getting into other work now… i'll loop back if casual conversation turns up anything interesting about the warnings debate.
- # [21:29] <capella> i have a beginners question im too embarassed to ask ...
- # [21:29] <tbsaunde> davidb: yeah, Im going back to reviewing once I stop catching up on irc
- # [21:29] <@davidb> capella: this is a safe channel
- # [21:29] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
- # [21:29] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
- # [21:30] <capella> who services the call on line 258 ...
- # [21:30] <capella> http://mxr.mozilla.org/mozilla-central/search?string=GetSelectedColumnsArray&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central
- # [21:30] <@davidb> can you bit.ly that?
- # [21:31] <capella> bit.ly == pastebin?
- # [21:32] <capella> it has to be an interface thing .idl ... why i can find the search ?
- # [21:32] <tbsaunde> capella: the impl on line 628
- # [21:32] <capella> c++ cant call a function with one vs two parms tho ??
- # [21:33] <tbsaunde> I assume the second arg is declared as optional, but haven't actually checked
- # [21:33] <capella> optional? c++? thats new to me
- # [21:34] <capella> i though itd be a mismatch error
- # [21:34] <capella> oh! i see it!!!
- # [21:34] <capella> function declaration thing
- # [21:34] <tbsaunde> capella: yes, C++ has optional args
- # [21:35] <tbsaunde> well, more default values
- # [21:35] <capella> default value kinda thing ... yes!
- # [21:36] <capella> i missed the "= nsnull" part yesterday night
- # [21:36] <capella> not sure i like that .... but will remember the trick
- # [21:39] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [21:43] <tbsaunde> khuey: ping, how much do you hate the use of VPATH in bug 762770?
- # [21:44] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
- # [21:44] * ChanServ sets mode: +o firebot
- # [21:45] <khuey> tbsaunde: why do you need it?
- # [21:46] <tbsaunde> khuey: I don't, surkov wants to put files under accessible/src/windows/uia/ instead of accessible/src/msaa/
- # [21:47] <tbsaunde> and aparently had linking trouble when he tried to just add another makefile
- # [21:47] <@hub> tbsaunde: I watch warning because it happens that sometime they show that my code is wrong... like calling the wrong method in Obj-C
- # [21:47] <khuey> I'd be interested to know what trouble he had
- # [21:47] <khuey> I'd like to avoid adding VPATH stuff
- # [21:47] <khuey> it makes the build slow
- # [21:47] <@hub> davidb: I was taking that changed approach to handle the enum values explicitely, definitely.
- # [21:48] <tbsaunde> khuey: well, I'm absolutely fine with that, but don't have much of a place to argue about build stuff with him from :)
- # [21:48] <tbsaunde> hub: I'm not sure I follow that, since *all* warnings may mean your code is wrong
- # [21:50] <@hub> tbsaunde: hence properly fixing the warning
- # [21:50] <@hub> hence me watching them
- # [21:50] <@hub> to be honest we should car about warnings as much as code style
- # [21:51] <@firebot> New Core - Disability Access APIs bug 765878 filed by dbolter@mozilla.com.
- # [21:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765878 nor, --, ---, nobody, NEW, ARIA abstract role to MSAA role mapping is spec violation
- # [21:51] <tbsaunde> hub: if it actually is an issue yes at some point, but we already have plenty of filed bugs for issues that actually bother people
- # [21:51] <tbsaunde> so I find a bit odd people decide they'd rather care about things that maybe might cause a problem
- # [21:52] <tbsaunde> than things that clearly are causing people problems
- # [21:52] <tbsaunde> hub: and caring about warnings means spending time and going out of your way
- # [21:52] <capella> tbsaunde++
- # [21:53] <capella> i learned a new trick today :)
- # [21:53] <tbsaunde> heh
- # [21:54] <@davidb> i would like access to BMO
- # [21:58] <tbsaunde> khuey: so, I guess I'll cc you, andmaybe Alex can explain what trouble he had
- # [21:58] <khuey> ok
- # [21:58] <khuey> sgtm
- # [21:58] <@davidb> can you access bugzilla?
- # [21:59] <capella> who me?
- # [22:00] <@davidb> anyone?
- # [22:00] <capella> yes i can - didnt you just file a bug?
- # [22:00] <@davidb> i did, and now i can't access it
- # [22:00] <@davidb> nor most of toronto
- # [22:00] <capella> i see it
- # [22:00] <@davidb> possible all of toronto
- # [22:00] <@davidb> ok thanks
- # [22:02] * Quits: victorporof (victorporo@C7255039.602BD6C.79933D60.IP) (Ping timeout)
- # [22:03] <@davidb> back
- # [22:03] * Quits: arky (arky@moz-7B0110AD.mv.mozilla.com) (Quit: Leaving)
- # [22:03] <capella> gremlins
- # [22:04] * Joins: victorporof (victorporo@C7255039.602BD6C.79933D60.IP)
- # [22:10] * Joins: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se)
- # [22:13] <capella> talk on #developers about bmo in and outs
- # [22:13] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
- # [22:16] <icaaq> is bugzilla down?
- # [22:17] <capella> in and out
- # [22:17] <capella> being tracked by bug 765670 apparently, but I can't confirm that because I can't load bugzilla
- # [22:17] <capella> says the #developers crowd
- # [22:29] <@firebot> hub@mozilla.com granted checkin for attachment 634093 on bug 755048.
- # [22:29] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
- # [22:30] * Joins: mike5w3c_ (MikeS@moz-294BE65F.xgsspn.imtp.tachikawa.spmode.ne.jp)
- # [22:30] * Quits: mike5w3c (MikeS@moz-B94F10FE.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
- # [22:30] * mike5w3c_ is now known as mike5w3c
- # [22:32] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [22:32] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
- # [22:48] <@hub> fyi, the "granted checkin" is "checkin+" that you should set when you commit a patch and [leave open] the bug for more
- # [22:53] <@hub> it is part of the latest change to the tree workflow.
- # [22:56] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Textual IRC Client: http://www.textualapp.com/)
- # [22:59] <capella> dang ... brb
- # [23:01] <@firebot> markcapella@twcny.rr.com granted checkin for attachment 634008 on bug 570322.
- # [23:01] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, mozilla16, markcapella, ASSI, random fail of image map (area) accessible creation
- # [23:01] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # [23:07] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [23:14] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
- # [23:29] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Ping timeout)
- # [23:36] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
- # [23:39] * Parts: richardschwerdtfeger (richardsch@A08B5B24.9DD4DBBF.6A7A197.IP)
- # [23:43] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
- # [23:45] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
- # [23:52] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
- # Session Close: Tue Jun 19 00:00:00 2012
The end :)