For some reason, I get this NPE if I enable that option and have debug on.  Line 228 is the for loop, but that array is initialized and final...  If I declare MOBILE_ICON_SPACERS as static in addition to final I don't get the NPE (compilers can optionally replace static members with the values themselves, maybe that's happening).  But it doesn't seem to do anything then.  I did grep for mSpacerView_Phone_1a in my SystemUI and its there (along 1b, 2a, etc).

03-17 17:52:40.920 E/Xposed  ( 1373): java.lang.NullPointerException: Attempt to get length of null array
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim.reduceMobileIconSpacing(StatusbarSignalClusterMsim.java:228)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim.apply(StatusbarSignalClusterMsim.java:165)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim$1.afterHookedMethod(StatusbarSignalClusterMsim.java:69)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:670)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.SignalClusterView.apply(Unknown Source)
03-17 17:52:40.920 E/Xposed  ( 1373):   at java.lang.reflect.Method.invoke(Native Method)
03-17 17:52:40.920 E/Xposed  ( 1373):   at java.lang.reflect.Method.invoke(Method.java:372)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim.update(StatusbarSignalClusterMsim.java:174)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalCluster.onIconManagerStatusChanged(StatusbarSignalCluster.java:483)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim.onIconManagerStatusChanged(StatusbarSignalClusterMsim.java:260)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.managers.StatusBarIconManager.registerListener(StatusBarIconManager.java:201)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalCluster.<init>(StatusbarSignalCluster.java:243)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalClusterMsim.<init>(StatusbarSignalClusterMsim.java:49)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.StatusbarSignalCluster.create(StatusbarSignalCluster.java:212)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.ModStatusBar.prepareSignalCluster(ModStatusBar.java:418)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.ModStatusBar.access$2900(ModStatusBar.java:70)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.ceco.lollipop.gravitybox.ModStatusBar$6.afterHookedMethod(ModStatusBar.java:598)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:670)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.phone.PhoneStatusBar.makeStatusBarView(Unknown Source)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.phone.PhoneStatusBar.addStatusBarWindow(PhoneStatusBar.java:4625)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.phone.PhoneStatusBar.createAndAddWindows(PhoneStatusBar.java:4621)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.BaseStatusBar.start(BaseStatusBar.java:510)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedBridge.invokeOriginalMethodNative(Native Method)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:656)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.BaseStatusBar.start(Unknown Source)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.phone.PhoneStatusBar.start(PhoneStatusBar.java:657)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.SystemBars.createStatusBarFromConfig(SystemBars.java:106)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.SystemBars.onNoService(SystemBars.java:58)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.ServiceMonitor.startService(ServiceMonitor.java:228)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.ServiceMonitor.access$000(ServiceMonitor.java:49)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.systemui.statusbar.ServiceMonitor$1.handleMessage(ServiceMonitor.java:73)
03-17 17:52:40.920 E/Xposed  ( 1373):   at android.os.Handler.dispatchMessage(Handler.java:102)
03-17 17:52:40.920 E/Xposed  ( 1373):   at android.os.Looper.loop(Looper.java:135)
03-17 17:52:40.920 E/Xposed  ( 1373):   at android.app.ActivityThread.main(ActivityThread.java:5312)
03-17 17:52:40.920 E/Xposed  ( 1373):   at java.lang.reflect.Method.invoke(Native Method)
03-17 17:52:40.920 E/Xposed  ( 1373):   at java.lang.reflect.Method.invoke(Method.java:372)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:901)
03-17 17:52:40.920 E/Xposed  ( 1373):   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:696)
03-17 17:52:40.920 E/Xposed  ( 1373):   at de.robv.android.xposed.XposedBridge.main(XposedBridge.java:117)

![screenshot_2015-03-17-18-15-03](https://cloud.githubusercontent.com/assets/355230/6699774/3eb0150c-ccd3-11e4-80c4-4515ca957bfe.png)
![screenshot_2015-03-17-18-19-40](https://cloud.githubusercontent.com/assets/355230/6699775/3eb18982-ccd3-11e4-8f6e-b3ab72e58295.png)


There might be a few miscolored things with the `button` style removed.


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/a01066dd1014b746887cc5df25a731d688b9a3aa/app.js)


Fix 284


done


It got removed in the final commit


Cite section or use hkdf


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png)


"experiments head" is used three times in these two lines, why not to assign it to a separate value with a meaningful name.


Why Ecto API will is defined here?


@mikebgx - Thanks for the update, and looks really good. I didn't wnat to split it into two parts, but would you be able to move parts of this thats relevant to IntelliJ into the IntellJ page just in the same folder, I'll accept this for now.


The application shouldn't depend on Exd


Never mind


I would move results.updated(...) to a separate line.


Why both? Ecto.Schema and Ecto.Model?


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/9a485f22-f89b-4287-aca1-07d287677cab/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/9a485f22-f89b-4287-aca1-07d287677cab/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/9a485f22-f89b-4287-aca1-07d287677cab/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/9a485f22-f89b-4287-aca1-07d287677cab/Chrome/elements.button.700px.baseline.png)


exd?


I think it should be run in the background with &:
sh producer.sh 100 127.0.0.1 2555 127.0.0.1:2555 application.conf &

The same applies for all consumer.sh


Maybe stop using the word "master" in relation to secrets.


Also have some extra nVidia cleanUP.


I think it should be run in the background with &:
sh producer.sh 100 127.0.0.1 2555 127.0.0.1:2555 application_round_robin_router.conf &

The same applies for all consumer.sh


On navbar-header this captures both the button and the collapsed-menu logo that swaps in at small screen sizes. The latter is covered already by Bootstrap, so the styling can be applied to the navbar-brand node only (the second child), which is the actual button.

It was not good that the site CSS file was not in an obvious location. A code reorganization is coming up that will make things easier to find, and maybe convey a better image of how the site operates.


Make it sense to have the ecto-tester helper on one place? Do not know exactly, in extra project or may be in ecto_migrate?


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/d57ddf67-85a1-4f3f-8856-e502e5586cad/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/05855557-706f-43eb-b9ff-5c6efd4ba196/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/05855557-706f-43eb-b9ff-5c6efd4ba196/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/05855557-706f-43eb-b9ff-5c6efd4ba196/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/05855557-706f-43eb-b9ff-5c6efd4ba196/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png)


Why not to use ecto_migrate there??


I suppose it depends on how much support you're looking to get. constantly transferring project owners is going to get old.


Yeah vjs-16-9 is redundant here. I meant it to be more of a pair with vjs-4-3, and those classes would be used by people who just want to add a css class in the html while vjs-fluid would just be used internally, but it could be simplified down. Let me know if that's still the way you'd go.

I can go down to just `vjs-fluid` (not `video-js.vjs-fluid`) if I add the width:100%, height:0 in the generated style element also, but not with vjs-16-9, or with vjs-fluid if that's going to be used like vjs-16-9. The generated style el adds default width and height values that would override the vjs-16-9 width and height since it's being created after the main stylesheet. The style el is already using a single class name so I can't make it any weaker.


Still needed?


Missing LF.


Hmm, I just added styling for the "+" on the main repo page... I didn't look any further, but I'll keep an eye out.


repo should be defined only in test


Nice KJ. I love your buttons though. Can you create an APK so I can try it out without having to re-compile everything or something?


"get(1).get.get(2).get(0)" - is exactly why I suggested to move it to a separate class. This is clear as mud


fore -> four


haha what? you can see this?

On Tue, Mar 17, 2015 at 4:33 PM, Ryan Balfanz <notifications@github.com>
wrote:

> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mackmcconnell/dotfiles/commit/aa927c50ea9d89eb0e7189877f5dd68915894b4c#commitcomment-10249190>
> .
>

--
Mack McConnell
@mackattaq <http://twitter.com/mackattaq>
Taster's Club <http://www.tastersclub.com/>


Det beror på om man vill stripa alla strings lika mycket, jag lade till två olika sorters strips:
En som tar bort all HTML-kod, och en som tillåter <i> bland annat. Kan vara användbart att ha två olika då det är kul att skriva i italics i kommentarer men inte i titlar.


You can use List(1, 2, 2, 2) instead, to create a list. From my point of view, this is more readable.


Mostly only high-level applications defines the repo, that means, that it shouldn't be defined in such ecto plugins.

And test should define own repo for test.


З@eβ:information_source:cb вообще коммит =(


Should Ecto.Subscribe.Adapter really know about all existing implementations? This is not extendable.


This definitely should be a function


Abri o issue #183 para discutir sobre essa questão.


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/01b2fd94-5c9f-47c2-a581-81ed122210fb/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/01b2fd94-5c9f-47c2-a581-81ed122210fb/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/01b2fd94-5c9f-47c2-a581-81ed122210fb/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/01b2fd94-5c9f-47c2-a581-81ed122210fb/Chrome/elements.button.700px.baseline.png)


Will you be submitting a PR for this? That would be great.


I don't really care, but we could cache this in the `hashid` module somewhere. As is, it will reload the prototypes every time `HashID` is instantiated. But usage of HashID as a Python module is still fairly rare, right?


Since you use version control system, why don't you just remove these classes completely?


any screen shot


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/7558759a-e965-4240-b70e-53aa4c186fa0/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/7558759a-e965-4240-b70e-53aa4c186fa0/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/7558759a-e965-4240-b70e-53aa4c186fa0/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/7558759a-e965-4240-b70e-53aa4c186fa0/Chrome/elements.button.700px.baseline.png)


Why are you using junit 3.x instead of junit 4.x?


I really wanted the vjs-4-3 and vjs-16-9 classes to be stand-alone. I did have the properties shared in a less function, `.fluid()`, but after doing that it felt overly complicated for so few styles. That's probably not what you meant though.


Alright, changed it!


Il va falloir faire une autre branche pour les tests de mouvements :back:


TeamCity Akka.NET :: Akka.NET PR Build [Build 158](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=275&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


This should be a function instead.


It's more idiomatic to use pattern matching here.


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-69.31%25-red.svg)


git add .


@as-cii You might want to +1 on this issue https://github.com/jashkenas/coffeescript/issues/2005


i++


Just det, ja nej då är det ju onödigt begränsat att kräma in det i samma funktion.


Why can't you assign a value to reducer at the same line you declare it?


I changed my mind. I"ll pull this as-is and do the formatting cleanups myself.


How can I do this with SassDoc? I didn't see anything on the site for it.


I would still need the break, as I want it to terminate if no exception happens but do … while is a good idea still


Please test this on all platforms and installation methods you support. I tested it on Python 2.7.8 running from the repo directory and as a local installation. Package it however you usually do and maybe even upload it to testpypi. And test it on Python 3 please.

I just discovered `pkgutil.get_data()` which made this less much less terrible than I expected. Apparently it's in the standard library and lets me just ask for the contents of a file placed relatively to a module.


Yes, and put misc.h in there for something else, so my thoughts of non portable are moot.


Why vertex has weight?


This commit covered until page 80 of the book.


So if it isn't opened first, it won't get re-encrypted?


nice!


shouldn't we set isTerminalBatch to true when we receive a COMPLETED queryState ?


Why not list?


Thanks Jeff. (I was sleeping! :stuck_out_tongue:) Had to double-check myself with MAP07, but it works fine...


@jtimberman I code reviewed this and changes look good to me. Sorry I cannot run tests though as I am on family vacation until late next week. But I think this is good to merge.


Wow cool, I will use them (and test them out); thanks Erik.


I think the alternative design could be:
class Graph {
  def vertices(): List[V]
  def edge(v1: V, v2: V): E
  def neighbors(v: V): List[V]
}

In this case you could eliminate Vertex completely, but it's definitely up to you.


The reason I missed this one was it had originally MEM_ALIGN_CACHE. I looked very carefully for SIMD, doh!

BTW anyhow this format is slower than Raw-SHA512 on all systems I know of, perhaps we should just ditch it.


It gets opened on creation, which is on demand. If there's an entry in there, it should be open.


It shouldn't have own repo.

There should be better solutions, there is a question in mailing list: https://groups.google.com/forum/#!topic/elixir-ecto/Yc7wcxU6j1M


iMPORTANT


@haio , thanks for your response…


You can use a function of type  T => Double instead. This is more functional and more idiomatic.


Seems, that at least this line is tabed, not spaced.


Ultima versão com prefab dos players antigos


TeamCity Akka.NET :: Akka.NET PR Build [Build 158](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=275&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:12:12
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


> &Enum.member?([:create, :update, :delete], &1) is short form


Good luck with that.


> BTW anyhow this format is slower than Raw-SHA512 on all systems I know of, perhaps we should just ditch it.

Same for SHA256 btw


After reading through the code I only now understood that map Map[T, Map[T, X]] is actually a graph. You should move it a separate class to avoid this confusion.


Thanks, please squash this commit.


Quanto ao `while`, realmente será melhor vir somente depois do `case`. Como está, para **cada** linha de todo o resultado ele executa o case, que por sua vez chama `zzdata` e `zzdatafmt`. Isso também vai economizar as chamadas à `zzpad` que hoje estão sendo aplicadas a todas as linhas desnecessariamente.


Also create a git tag :-)


what's undefined for? Add a param comment


Just to let you know, though, every singe block requires IC2. You're going to have a lot of work to do. Also I'm not going to help you.


Why not simply to_string?


This looks good - just a few points.

Remove the console.log statement and use $log if you need to as IE8 hasn't got console.
I would change the name of the recursive function to a more descriptive name like findParentForm.
I'd also do a undefined or null check on the parent element in the recursive function just in case.


This code is only called from "build". Why did you move it to an inner function?


That's an interesting note. Could we lay out a full use case behind when that would be used? The video element will actually parseInt any dimension value you pass to it. e.g.
```
videoEl.width = 300.9;
var w = videoEl.width;
// returns 300
```
So I would say we should stick to the same unless we can prove a good use case for fractional values. With this PR you can now override any of these values with CSS, which I would assume is the main way people would want to style the player, and the width/height attributes would be more for simple use cases.


tabs(or too many whitespaces)?


yes @byorgey, sorry , besides that, is the rest okay? Let me know if the rest is okay to remove this file from the commit and to squash all the commits.


what's your OS version or ffmpeg version? I have to make sure QtAV can compiled with ffmpeg >= 1.0 and libav>=9. channel_layout.h is missing in ffmpeg versions. If it exists, it is included somewhere.


I think this should be 'message' ?


I won't be able to make it to the meeting this time. I have my parents
visiting from Spain and my mother in-law just a broke a hip. Are these the
infamous Ides of March? 🙀

On Monday, March 16, 2015, Scott Windsor <notifications@github.com> wrote:

> Yup, we're on Ch 16 this week [image: :thumbsup:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SeaRbSg/little-schemer/commit/156bb1b5d2208c6a6a4d7339c77a265d5169c290#commitcomment-10227807>
> .
>


This setting affects the http, https and admin listeners. It'd be better to name the config 'http-server.accept-queue-size'


Thanks @mcm-ham
The reason to replace alias was the updating count of offers shown in tabs of my list page.
Previously it was not updating count of reviewing offers when reviewer starts reviewing offer.
As you have said in above comment filterBy("reviewedBy.id", currentUserId)
Will the filter work with reviewedBy.id? as it needs proper attribute name right? or we need to add filter function here?
Am also not completely sure, I will try with above. Thanks.


This line. Why did you remove it?


Why did you end up removing the automatic data-slick call? I may have missed a conversation somewhere.


You're defining these as modules as program globals. If you do it this way (since there are multiple CPP files), there will be multiple instances of each and will have multiple copies.

Inventory, Cashier, and Report should only ever be called in Main. If MainMenu needs to use it, you need to change its constructor so it can hold the reference instead.


asd


I inserted values like
42601,46598,1
or 42601,79436,3
but the action bar has always empty spell click area..
the  npc_spellclick_spells  wiki is not really helpful, why should I insert values??

on offi sniff I have after summit npc 42601 this packet:
ServerToClient: SMSG_PET_SPELLS (0x095A) Length: 68 ConnIdx: 0 EP:  Time: 07/29/2014 22:33:45.000 Number: 287895
[0] Spell: 79425 (Reaping Blows)
[1] Spell: 79430 (Rocket Boost)
[2] Action: 0
[3] Action: 0
and so on......
Family: 0
Int40: 0
Int24: 599874
Int20: 257
Guid: Full: 0xF150A66900D51DA7 Type: Vehicle Entry: 42601 (Overloaded Harvest Golem) Low: 13966759

I was try: init actionbar or initpetbar,
and use
AddSpellToActionBar(...) or SetActionBar(...)
but the spell click area is always empty..


It seems we need two more jars (see https://github.com/droolsjbpm/kie-wb-distributions/blob/master/kie-wb/kie-wb-distribution-wars/src/main/assembly/tomcat7/README.txt#L60)

The changes are trivial, so I pushed them directly: https://github.com/droolsjbpm/droolsjbpm-integration/commit/9cb71fd7cf7d4a2e70f7334ac99dfd3eb5fa6d30


This function doesn't exist.


aa


rax retry tests


Prix_ajusté*qte_vendu? (il faut les ajouter a la classe produit) À moins que j'aille mal compris le but de la méthode.


Sorry, I'm not familiar with deb tools. It works. But I don't know how is qmake invoked, can you tell me more details?


hi,
i wondered how to do the 'net surgery' since you make the function Net<Dtype>::params() to be const (not in python but c++)? is there another better way to replace the model parameters with a pre-trained one?
best wishes


Sorry, just sat down. line 657 is calling a file (actually, the directory) that doesn't exist. ```~/unified-node-open-mining-portal``` should be ```~/unomp```


Try using `wiki_collaborator_path(@wiki, user)`


It looks like this function is returning a queryset some times and an HttpResponse others. The latter case doesn't appear to be expected though, as the results of `loan_originations` is iterated over in the next function.


Oh no - I hope your mother-in-law is okay.

No problem, we'll be here when you have time again.

> On Mar 16, 2015, at 8:02 PM, Javier Soto <notifications@github.com> wrote:
>
> I won't be able to make it to the meeting this time. I have my parents
> visiting from Spain and my mother in-law just a broke a hip. Are these the
> infamous Ides of March? 🙀
>
> On Monday, March 16, 2015, Scott Windsor <notifications@github.com> wrote:
>
> > Yup, we're on Ch 16 this week [image: :thumbsup:]
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/SeaRbSg/little-schemer/commit/156bb1b5d2208c6a6a4d7339c77a265d5169c290#commitcomment-10227807>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


... Não.

Se você fizer isso, a mudança de status do protocolo vai ser muito trivial, digamos assim, e a idéia é que a mudança de status tem que ser bem deliberada pelo operador. Além disso, eu tou sentindo que vai ser pedido que seja gerado log da mudança, ou o operador vai ter que preencher o motivo da mudança toda vez que ele mudar de status do protocolo. Se a mudança de status for uma coisa trivial, o operador vai ficar na cabeça que é uma coisa desimportante, e não queremos isso.

Sem falar de que se a pessoa estiver com o filtro de status, (de NOVO, por exemple) ela vai perder o protocolo da vista dela, e nunca mais vai ser encontrado.


thanks for the update! I was curious why it was showing Turkish Lira as ₤.

For the Turkish guys i'll add here a comment explaining it.

> Bir hata nedeniyle Gnu Cash Türk Lirasını (TRY), ₤ sembolü ile göstermekte idi. Yukarıda belirtilen        > dosyada ki değişikliği yapdığınız takdirde olması gerektiği gibi ₺ olarak gösterecektir.


Commit revertido pelo commit 185608c


updated in master


Just a style comment:
```(self.skills & current_employee.skills).count```
is more pleasing to the eye due removal of superfluous paren and use of whitespace


r+


Will the util library too big to be browserified and fed to client-side?


Isso, claro, sem falar que temos o pusher que é justamente não precisar recarregar a página.


Please use Array.some()
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/some


:cake: for both @eriol and @ralphbean


Do you know what happened with this URL? Cannot find AWS::SQS:: QueuePoller and  AWS::SQS::Queue.new anymore on the official docs


Desole j'ai regle ce probleme

2015-03-16 23:09 GMT-04:00 Chuck <notifications@github.com>:

> Prix_ajusté*qte_vendu? (il faut les ajouter a la classe produit) À moins
> que j'aille mal compris le but de la méthode.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbarrette/flaming-flamingo/commit/88189332be558ea2b564352189bc276985548b66#commitcomment-10227937>
> .
>


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1027
success: http://build.servo.org/builders/linux2/builds/1027
success: http://build.servo.org/builders/android/builds/1061
success: http://build.servo.org/builders/mac1/builds/1243
success: http://build.servo.org/builders/mac2/builds/1243
success: http://build.servo.org/builders/gonk/builds/214


rax retry tests


I get 6 failures when I run this.  I eyeballed an error real quick... labels are overlapping.  Getting there though I think.  I thought about this and I really don't think I want to allow grouping per se, but you might very well convince me otherwise!

I was thinking of our original cases, that we wanted and it was beginning and end of line, empty line right? I think its fine by me if they're one offs in the code.  They don't have to be part of the regexes I mean per se.  What do you think?  Also, beginning of line is useless unless there's nothing else on that line.  I think first word + the home key suffices.  No one *looks* for space to the left of the word.. If they know a line number, they can use built in line jumps.  Really we want to take the user to where they want to go right? (Rather than provide every jump location that is).

So beginning of line on empty new lines and end of lines everywhere else.  I want to add in the ability to match on curlies for c like languages too.. I'd be ok with all of the above being one offs though.  Thoughts?

If we can catch the patterns for *most* programming languages and *most* spoken languages that's the goal right?

Anyway, if you can get it with no errors or with reasonable tweaks to the tests (maybe some broken, although I can visually see issue right now in some cases, so needs further tweaking) maybe we will continue down the group by path.  I bet you and I could do this in less lines with the code change though (of course we'd have to tweak the tests to expect for new added jump targets too).


tag is added at the start of filename.


Thank you very much!


@shelbys can you take a look again?


initially i=2 was written for . and .. folders (to avoid infinite loop)
not required now


dh_auto_configure checks if there's a *.pro file, and in that case it uses qmake.

However, in a next commit I've reverted this, since it seems that it causes troubles when linking.


fast-forwarding master to auto = 990a965d


hello


saw approval from jdm
at https://github.com/nnethercote/servo/commit/fa9ca206efefea850ef1d8c8b43513653ff07507


merging nnethercote/servo/measure-display-list = fa9ca206 into auto


nnethercote/servo/measure-display-list = fa9ca206 merged ok, testing candidate = f0936209


Emilio


I agree. I'd be happy to go without match groups. I think it should be a given that every line gets a token at the first and last characters on each line and one on every blank line. This could be hard-coded or even toggleable via the settings screen (defaulting to on).


I'm in Ubuntu and I'm using libav actually. I'm getting compilation error in ubuntu >= 14.10 without including channel_layout.h.

It seems that it's only included from libavutil/audioconvert.h here (not sure about pure ffmpeg), so we probably could use that header if it's shared in both libs.


Thanks, will fix.


Improved coding style in this method :)
My only ding would be that this code relies on the semantics of the ```push``` method to determine what the method returns. In all execution paths a call to ```push``` is the last expression evaluated so your method always returns whatever ```push``` returns, specifically its receiver. While this works fine, the reader needs to know that ```push``` behaves that way in order to know that ```elements``` will be returned. I would probably be more explicit by having either ```return elements``` or more tersely ```elements``` as the last line of code in the method.


- Colored output on screen
- Code improvement
- Metatags Remove Result


Hi Louis. this is a comment


Yes it could be. I tried it briefly, but what I tried had a negative impact on sysimg build time, since it actually ended up needing many of the ASTs that were removed or not saved. By the time we're saving the image, they're much less likely to be needed again.


Lol!

Thanks a lot for your swift addition of f2fs. I was looking at ~60 file changes as opposed to your 8 to implement it, I'm very happy you were able to.


Updated.

A side note is that this unfortunately means that if you've upgraded the master and half your nodes, and things start going south, you need to downgrade all your nodes before you can change the master back. Might be fine, but maybe still not ideal.


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (639 of 639 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-4.23%25-green.svg)


@IngageKrizJ we do want to use rendered content.


Double check this works. When I tested your Task * app on Heroku, the "My Requests" page had no relevant skill info displayed even though relevant skills were indeed assigned.


I see that. Moved them to int main(), now cash.findISBN
or cash.anything gives error.
Do I need to make it hold the reference? Ummm how do I do that? :cold_sweat:


ok, yeah that would be bad. perhaps we could just store the header of the lambda info instead of just the return type or the whole tree? if all variables are type-stable in the function, it would be pretty cheap to recreate the full tree and do the other optimizations (inlining, tuple-elision, etc).


imap


LGTM


Ok, thanks, i wasn't sure... I put a comment in the file with this question
as well.  I'll change it back.
On Mar 16, 2015 11:41 PM, "Dan" <notifications@github.com> wrote:

> @IngageKrizJ <https://github.com/IngageKrizJ> we do want to use rendered
> content.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/crdschurch/crds-angular/commit/f0ff36133ea850f1b5f9d9d8194fe60bc271bd34#commitcomment-10228142>
> .
>


Maybe a link explaining where this list comes from?


@ale-php


Does this work too? (no seriously, I'm not sure so better check)
```
def interest_count(user)
  (self.relevant_skills & user.desired_skills).count
end
```


`java_import(gateway.jvm, "...")` does not make any error when there're no package available to import.
So i think it's safe to keep here to provide the same behavior with `spark/bin/pyspark` repl shell.


Can you eliminate the use of register `r3`?


Nice change!


Consider same rewrite as for ```interest_count``` method.


81606030-Cooling-airflow-rate-being-used-in-heating-mode-for-variable-speed-cooling-coils-(Github-#4564) (rraustad) - i386-Windows-7-VisualStudio-12: OK (1655 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


Did you mean: `comments: [comment-schema]` ?


@luccasmaso Sorry, that URL is forward looking. It will be valid once the SDK is tagged and released with this change. If you'd like to preview these docs, you can git clone the repository, bundle install, and then `rake docs`.


from `[DefaultValue]` attributes


Thanks for the quick fix!

On Mon, Mar 16, 2015 at 3:43 PM, James Pearson <notifications@github.com>
wrote:

> Sorry about that. 1da986e
> <https://github.com/reddit/reddit/commit/1da986efa987cffae3d3ee3308871239f3daad5b>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reddit/reddit/commit/9c6aa71f3c71adb7ea210440f6f43cdcfa02b439#commitcomment-10224824>
> .
>


Очень ненадёжный способ удалить объект. Когда мы внезапно поменяем набор полей этого объекта, программа будет иметь неопределённое поведение (фактически будет падать), т.к. ячейка удалится, а объект из базы — нет.


Some recovery directory entries get created before you dynamically add any exports.  Overall, it should NOT work as you probably noticed in your later testing...


It looks good


Speed is actually not affected. Just seems like it since the size changes :)


Perfect! Thanks.


How do these tests work?
why ```user2``` ?


Is it possible to eliminate the use of register `r7`?


:clap:


https://github.com/TrinityCore/TrinityCore/tree/3.3.5/src/tools/mmaps_generator/ has been like this for 9 months now. So if you haven't noticed anything wrong these 9 months, that means it is working OK.

If you need to know how CPU is used, you need either to check while it is running or read the source files:
https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/tools/mmaps_generator/MapBuilder.cpp
https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/tools/mmaps_generator/MapBuilder.h


@val99erie @jcoyne - I don't think you'd ever have the same name URI associated with one record as a personal name and another record as the corporate name.  Couldn't we simplify this by modeling Agent as having a foaf_name and a name_type (predicate rdf:type, range:
http://www.loc.gov/standards/mads/rdf/v1.html#ConferenceName,
http://www.loc.gov/standards/mads/rdf/v1.html#CorporateName, http://www.loc.gov/standards/mads/rdf/v1.html#FamilyName,
http://www.loc.gov/standards/mads/rdf/v1.html#PersonalName)

I think name_type is substantially different than role in that one agent could conceivably have different roles for different object, but I can't envision a case where an agent would have a different name types for different objects.

Also, reading through the code, I find the decision to collapse family and conference to 'group' and to use the the class name 'contributor_class' confusing.  In MARC, MODS, and MADS, where these usages come from, there's a pretty clear sense of four name/agent types.

Would you two be open to a PR if I made changes along these lines?


<identifier>IronHigh</identifier> <!-- Needs another tab so it fits syntax properly -->
 <substring>Blacksmith's Tribute Iron</substring>


I haven't used FactoryGirl before, but I get the idea: fixtures. Nice. Did you really write the tests first and then add the user validations to fix the failing tests? :)


Don't add the generated files such as ` lab-1i/fibseq` in GIT repository.


Hmm... the 80 byte default was merged into master, but didn't make it into the 0.10 release.


Thanks for the comment. Changed the property name.


This context is named incorrectly.
It should be something like `formatting objects inside other objects`.

Suggestions?


@jacshfr do we plan to set up the GTM Container ID as a settings property, like they formerly used with `settings.GA_ACCOUNT_CODE` (as in previously line 11)?


同上


ready to ship after discussion.


I am unsure about why the default value is passed in both on creation and access of the property.
Is this a documentation mistake?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1028
success: http://build.servo.org/builders/linux2/builds/1028
success: http://build.servo.org/builders/android/builds/1062
success: http://build.servo.org/builders/mac1/builds/1244
success: http://build.servo.org/builders/mac2/builds/1244
success: http://build.servo.org/builders/gonk/builds/215


The problem is that you do not decrement the length in your ```DeleteElement``` function. The function works correctly, however, after that function executes your code will actually crash at ```PrintLast``` because it assumes your function has a bigger length than it actually has. (function actual length = 4, but because you forgot to decrement it, it has 5 there)


The elements of the `vector` are not `const`; this change has no effect on editing model parameters.


Furthermore, you now have a ```prev``` field in your structure. Why not start from the end and go x positions from right to left? why keep the same code from last assignment when the actual reason for having a ```prev``` field was to ease up this function?


fast-forwarding master to auto = f0936209


<!DOCTYPE html>
<html lang="en">

	<!-- a head for all the heady/conceptual/meta stuff -->
	<head>

		<title><!-- your title goes here --></title>

		<!-- this imports p5  -->
		<script language="javascript" type="text/javascript" src="p5/p5.min.js"></script>
		<script language="javascript" type="text/javascript" src="p5/addons/p5.dom.js"></script>

		<!-- here's your sketch! -->
		<script>
		var lineNumber = 0;
		var maxLines = 400;
    var columnNumber = 0;
    var maxColumn = 100;
    var columnPercent;
    var linePercent;
    var progress;
		var output = '';
		// we're in javascript land now!
		// (notice the different style of comments)
		// here is where you declare your vars, like:
		// var one = 1;
	 /**
	 * Setup - this is where you get things ready before running the loop
	 */
		function setup() {

			// this tells p5 that we are not using the
			// `<canvas>` element (more on that later)
			noCanvas();
		}
		/**
		 * Here's the draw loop! This is where all the fun stuff happens.
		 */
		function draw() {
			// this will continue to loop
			if (lineNumber < maxLines) {
        linePercent = lineNumber / maxLines;

				var htmlTag = "div";
				var cssClass = '';
        var lineRemainder = lineNumber % 5;

          while (columnNumber < maxColumn) {
            columnPercent = columnNumber / maxColumn;
            progress = (linePercent + columnPercent) / 3;
            if (progress < 0.1) {
            output = output + "|";
          } else if (progress < 0.3) {
            output = output + ":";
          } else if (progress < 0.5) {
            output = output + "|";
          } else if (progress < 0.8) {
            output = output + ":";
          } else {
            output = output + "|";
          }
          columnNumber = columnNumber + 1;
          }

        var element = createElement(htmlTag, output);
				if (lineRemainder == 0) {
          element.style("color", "#000080");
				} else if (lineRemainder == 1) {
          element.style("color", "#6B8E23");
				} else if (lineRemainder == 2) {
          element.style("color", "#FFA500");
				}	else if (lineRemainder == 3) {
          element.style("color", "#FF4500");
				} else if (lineRemainder == 4) {
          element.style("color", "#DA70D6");
				}

        output = '';
				lineNumber = lineNumber + 1;
        columnNumber = 0
			}
		}
		// now leaving the land of javascript...
		</script>
		<!-- back to HTML -->

		<style>
			body {
				padding: 0;
				margin: 0;
				font-family: 'Helvetica Neue', 'Helvetica', 'Arial', sans-serif;
				font-size: 200%;
				color: black;
			}
		</style>

	</head>

	<!-- a body for the meat/data/content -->
	<body>

	</body>

</html>


NICE SPACING BRAH


if ```sent->len++;``` is common to both branches, why not pull it out and have it as a last (common) statement of the ``addFirst``` function?


same as above


"dragdrop is not defined"


Ah, creo que más bien arreglé el filtro principal no el de datasets D:


what's the effect? @_@


30411/Sporis A. Ioan Catalin/Doubly linked list/main.c

here you should be having
```
if
else if
else if
```
instead of just ifs.


saw approval from glennw
at https://github.com/nnethercote/servo/commit/2b10f6e7ea1c5a6bd6adcb5ed845c7bcb94f8e35


merging nnethercote/servo/only-one-quit-event = 2b10f6e7 into auto


nnethercote/servo/only-one-quit-event = 2b10f6e7 merged ok, testing candidate = 2281bca8


``` Right click -> Format use AStyle```
anything else looks good


I believe the intent was to have things like damage formulas be short calc scripts. Which we may need some day but you are right, we can bring it in via nuget when its needed. (Probably for sample combat impl). Sorry for line notes, reviewing on phone is cumbersome. :)


missing `,`


앞으로 파일 수정할 때, 맨 끝에 한 줄 더 띄우는 습관을 가지자


please upload the whole projects next time, including their containing folders + the ```.cbp``` files


container 클래스를 지우자


Would you consider further instruction scheduling, especially for dual issue?


Strange I just tried it using alias and the count updated for me. Yes `reviewedBy.id` should work, I've been using it before but just did a quick test to make sure and it worked. That said there is one problem with my code, if a new offer is added the property won't update since I'm not observing the store to see if the offer array has changed. So I think it actually needs to be this:

```js
allOffers: function() {
    return this.store.all("offer");
}.property(),

reviewerOffers: function() {
    var currentUserId = this.session.get("currentUser.id");
    return this.get("allOffers").filterBy("reviewedBy.id", currentUserId);
}.property("session.currentUser.id", "allOffers.@each.reviewedBy")
```

You don't need to worry about observing anything for the `allOffers` property since it returns a RecordArray which supports live updating (this is why `filteryBy` can't be used in route because it only returns a normal array). `store.filter` does return a RecordArray but from memory when I tried it I don't think live updating works more than one level deep, not sure why given that it's a custom function.

Alternatively to avoid the user/userProfile conflict I see happening, we could just add userProfile as a property to user so it then becomes `session.get("currentUser.profile.mobile")` and `session.get("currentUser.reviewingOffers")`. This seems the cleaner approach and am happy to make the change if you like.


Yes, this is a documentation mistake.  The default value is only passed in to get().


30414/Stoica S. Razvan Cosmin/Assignment 2.2.cpp

why not just keep a tail variable somewhere? (in a sentinel maybe?)


good work @sunil-sharma - keep this code but refactor valid_host? to look for a specific server header.


Good! Please do mention the ratio of your improvement.


interesting way of doing it


@sunil-sharma let's set a global server header instead


just write describe, no need for RSpec in front of it.


30414/Stoica S. Razvan Cosmin/Assignment 2.cpp

you're missing the ```length``` field


I doubt this is needed. Try and get rid of it and see if this still passes


this is completely wrong, please explain why all the overhead is needed? it should be almost as simple as the previous assignment's ```AddLast``` function


is there a test for that?


also, some better names for your variables will be nice.


@akshaydorwat probably makes sense to make a .gitignore to keep others from accidentally adding these to the repository


Oh *bleep* you're right :( https://github.com/bitcoin/bitcoin/blob/0.10/src/script/standard.h

Thankfully anything Core can currently broadcast fits in 40, but shoot that means it'll be 0.11 before that gets into mainstream clients and even longer before miners adopt it :(


You might consider pulling this out in to a partial. Its complicated enough (thanks bootstrap!) that its annoying to look at in the context of the rest of the app layout.


r=mbrubeck.  Can merge whenever ready.


passing a list as a parameter -> very nice and useful


@joshiste `enabled` should be `enable` or the above documentation needs to be fixed.


however in this assignment you should have used a sentinel as well


In order to avoid having to do this, lets just validate that the start date is always a monday.


very nice


@lapetus28 David -- did you do anything with this PR in terms of getting it merged in to the "core" Swagger-UI stuff? I'd be very interested in getting Swashbuckle/Swagger going with OpenID Connect...


Great job, really!


Lets talk about this tomorrow.


工期地


30411/Grigoras C. Alexandru Mihai/Double Linked List/main.c

you are missing your sentinel


30411/Fericean G. George Vlad/Double Lists/main.c

I think ```char string[30];``` should be sufficient.


Does this work for vanilla `NSViews`? I know in iOS I often am laying out subviews that aren't `YOView` subclasses (UILabel for example). Wouldn't want to break OSX compatibility with views in the hierarchy that aren't `YOView` subclasses.


@joshiste As a side note, `enabled` is commonly used but `enable` is used here. I guess to change `enable` to `enabled` is better for consistence in the next major release.


You should probably add `-` to the list.


Usando SharedPreferences en lugar de una variable estática.


The annotations for passing tests should actually be removed, both the [foo] and the following line.


Apparently this can do more than just hang: it will OOM the process.


Your problem is this function, not the main.
What happens if your list is empty?


I'm not entirely sure if String types will be implicitly declared as of C++ 14 or not.


30411/Pomian G. Gheorghe Aurelian/QueueProb/main.c

you can eliminate this whole branch


Is this backwards - or am I looking at this too late


you can eliminate this branch by adding ```tail=head``` as a last statement of the previous branch


More evidence that I should stop looking at code this late


What was the difficulty when trying stuff out with the sentinel?


thanks for correcting my grammars @falconflame :D


oh, I mean you should use symbols instead of magic numbers.
so should be GPIO_ACTIVE_HIGH


Using "extern" so I can allocate and explicitly control the way these prototypes will function.


I'm just going to use the standard memory allocators, this saves me tons of time!


long lines


this is wrong. if the list is empty this will crash


same here


haha


again,thanks alot brother !! @falconflame


but what if the head/tail do not contain the code you are searching for?


This is actually really good, I'd just put everything but the first sentence in the remarks.


also this code doesn't even try to search for the actual ```code``` you want the node to contain


Yes, good idea.


empty button text


You have to change the constructor of `MainMenu.cpp` so it takes an argument for the reference, then store it as a member variable inside the class. Look at how the Cashier constructor is made


if your k starts from 0 I think you might want to go up until ```<x``` not ```<=x```


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1148 of 1148 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.61%25-green.svg)


"debug" methods should be left out of the final code


you can remove this branch


Santinela? ce păzește?


yeah... it will cause NSViews with autolayout to update and I think thats ok..

http://stackoverflow.com/questions/4676498/does-nsview-have-anything-analogous-to-uiviews-setneedslayout-layoutsubviews-me#answer-12741741


When we used to createInferenceContext in this function, we initialized it with an empty array for inferredTypes. So every call to inferTypeArguments had a fresh array. Notice now that we call this inside the `while (true)` loop in chooseOverload, but we createInferenceContext outside the loop. So on the first iteration, the call to inferTypeArguments won't need to clear out the type arguments, but on all subsequent iterations, it will have the values from the previous iteration. I could also move this clearing logic into chooseOverload toward the bottom of the loop.


@kazegusuri https://github.com/GoogleCloudPlatform/kubernetes/pull/5534, since I need it for one of my cls as well


added


It cannot be undefined. It's only undefined when originalCandidate.typeParameters is undefined. But here we are guarded by `if (candidate.typeParameters)`, and notice from the line just above that, `candidate = originalCandidate`


Cool! Ship!

On Mon, Mar 16, 2015 at 9:51 PM, Gabriel Handford <notifications@github.com>
wrote:

> yeah... it will cause NSViews with autolayout to update and I think thats
> ok..
>
>
> http://stackoverflow.com/questions/4676498/does-nsview-have-anything-analogous-to-uiviews-setneedslayout-layoutsubviews-me#answer-12741741
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YOLayout/YOLayout/commit/349addfaa7866c9eadf7e861d4dab3d68631d51a#commitcomment-10228613>
> .
>


Instead of doing this, I would return the interface from the New method.  Either way, someone can cast to your public methods, and returning is the standard convention in other parts of our code.


Не вижу повода делать этот импорт.


@rpg777 this broke the build (missing closing parens).

Weird thing is that all the dashboards are so broken you can't tell :-(

@axelstudios @kbenne


Am incercat sa fac dar imi crapa incontinuu am stat vreo 2 ore la el sa incerc cu santinela


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1029
success: http://build.servo.org/builders/linux2/builds/1029
success: http://build.servo.org/builders/android/builds/1063
success: http://build.servo.org/builders/mac1/builds/1245
success: http://build.servo.org/builders/mac2/builds/1245
success: http://build.servo.org/builders/gonk/builds/216


The semantics are different than my method:
_.defaults doesn't do more than the first layer
_.merge merges in the other direction from deepMerge - e.g. deepMerge(overrides, original) == _.merge(original, overrides)

I've mapped things over, but still running into some issues, so must be missing another semantic difference.  Should hopefully have it worked out soon.


we should add passable --options for everything but password and maybe some validation


good catch, will fix; note explicit guard support was dropped late in the bufrw 0.9.x cycle, and the above is a hold over... will convert to supported path (a "call" field like frame uses) in both cases.


Рекомендую использовать следующий стиль именования значений перечисления (хорошо выглядит в авто-дополнениях):
```
typedef NS_ENUM(NSInteger, StorageMethod) {
    storageMethodFile,
    storageMethodDatabase
}
```


@Garagoth, all of our test methods start with a word "should".
Can  you please rename them so that they had the same style?
Please, look through the code for plentiful examples.


I feel that that check belongs higher up outside of the read/write layer, but sure let's add that as a check at the req/res send layer?


Note this used to be `break`. I think it should have been `continue`.


Sweet emacs dotfiles yo :)


Why do we need Network access?


good events


fast-forwarding master to auto = 2281bca8


Чтобы не обманывать людей которые читают твой код, следует использовать функцию `NSLocalizedString()` для локализации строк.


good catch, I managed to drop that ... I'll add it back and a test to catch it at his layer (call req / res).

Also note, as noted on init, the streamGuard in this current form is a noop as the guard API went aay.


81606030-Cooling-airflow-rate-being-used-in-heating-mode-for-variable-speed-cooling-coils-(Github-#4564) (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


81606030-Cooling-airflow-rate-being-used-in-heating-mode-for-variable-speed-cooling-coils-(Github-#4564) (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


81606030-Cooling-airflow-rate-being-used-in-heating-mode-for-variable-speed-cooling-coils-(Github-#4564) (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


81606030-Cooling-airflow-rate-being-used-in-heating-mode-for-variable-speed-cooling-coils-(Github-#4564) (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


Since :bind-keymap doesn't add to the `commands` variable like :mode, :interpreter and :binding, it needs to be explicitly checked here when we set the `defer-loading` flag.


Thanks =D. Just started getting back into emacs and realized I should build up some new dotfiles. Next up, splitting the init.el!


Okay. I will keep the I/O peripherals in the subsystem itself.
yes. It is the only option to export the clock to the toplevel.


LGTM.


I've tested on ubuntu 14.10 and no build error. If you really need it, checkout the latest code and add a check around it

    #if !FFMPEG_MODULE_CHECK(LIBAVUTIL, 51, 73, 101)
    #include <libavutil/channel_layout.h>
    #endif


LGTM


@vladimir-kotikov - Can you please review?


r+


В коде очень много явных приведений `id` к типу и это выглядит некрасиво. Варианты решения: 1) не использовать простые словари/массивы, а сделать классы-контейнеры для нужных типов; 2) завести промежуточные переменные, тогда приведение типа не требуется:
```
NSNumber *strengthNumber = [localizedPasswordStrengthOptions allKeysForObject:option].firstObject;
PasswordStrength passwordStrength = strengthNumber.integerValue;
```

Использование `firstObject` предпочтительнее доступа по индексу, т.к. никогда не вызовет исключение. Доступ по индексу в пустом массиве вызовет исключение.


@tchoulihan , please remove commented out code.


Does Opera really need its own `manifest.json`? Seems identical to Chromium's `manifest.json` except for the lack of `options_ui`.


Рекомендуется всегда писать код максимально однородно по стилю, в том числе указывать атрибуты свойств в одном (по сути любом, но одинаковом) порядке.


Это событие не `touch`, а `tap`. Отличие в том что первое происходит сразу по началу касания, а второе — только после отпускания кнопки с пальцем в пределах самой кнопки.


测试代码评注


@tchoulihan, is there a reason you are purging data in what is essentially a "getter" ?


测试行评论


saw approval from jdm
at https://github.com/doublec/servo/commit/618142fac7a580071f758333f91afc683e70f6d7


merging doublec/servo/view_source_protocol_and_plain_text = 618142fa into auto


doublec/servo/view_source_protocol_and_plain_text = 618142fa merged ok, testing candidate = 472f2d34


testtt


@meteviasa I tried to limit the frames here as well, no result


@Garagoth, should we test if these attributes are actually dirty? Also, what is the value to pass the "dirty" information to a new model?


dasdda


@meteviasa Hey, do you mind looking at this when you get the chance? I have been trying a bunch of things to try to change the number of frames I play (to manipulate pitch), however, I just can't seem to manipulate the frames. Do you have any idea where I should do this? I tried to do it in CWaveInstrumentFactory and in CWavePlayer. I just wanted to get something working, so I tried to use limit the number of samples like this:


@axxwell contact me on s.k.y.p.e rbalaquiao i have channels checker for Secured Channels


The precedence of > and < operators should be lower than + and - operators.


Имена переменных следует начинать с маленькой буквы.


Doing this just gives me more errors.

On Mon, Mar 16, 2015 at 9:48 PM, Dylan <notifications@github.com> wrote:

> You have to change the constructor of MainMenu.cpp so it takes an
> argument for the reference, then store it as a member variable inside the
> class. Look at how the Cashier constructor is made
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dooks/stacks-overflow-group/commit/f2209388ede5304b75b116dcaae1e1a8388fc87e#commitcomment-10228596>
> .
>


90298376_ResultsViewerUpdate (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Fixed, but still get an error on every line with
cash.something

On Mon, Mar 16, 2015 at 10:25 PM, James Chantler <chantler.james@gmail.com>
wrote:

> Doing this just gives me more errors.
>
>
> On Mon, Mar 16, 2015 at 9:48 PM, Dylan <notifications@github.com> wrote:
>
>> You have to change the constructor of MainMenu.cpp so it takes an
>> argument for the reference, then store it as a member variable inside the
>> class. Look at how the Cashier constructor is made
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dooks/stacks-overflow-group/commit/f2209388ede5304b75b116dcaae1e1a8388fc87e#commitcomment-10228596>
>> .
>>
>
>


We need to consider IPv6 for this, unfortunately. I'd suggest either [looking at these regexes](http://stackoverflow.com/a/17871737/4552952), or using net.ParseIP() and aborting if it is a valid IP.


d


Zoey, I have been using Maven for years on all my L2J based projects. I can make a project model (in patch) for you if you wish, and the only thing you have to do "to make project ready" is to split LS-GS-Commons properly which is 5 to 15 min for a dev like you. :)


90447068-BalancedExhaust-Issue4711 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1648 of 1649 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


Check how "XStatus" and "XStat" and "XKnownList" classes work and you will understand what I am speaking about. :)


Another suggestion:
if a {
  if b {
  } else {
  }
}


r=jdm


Этот коммит неудобен для чтения — слишком большой. Следует выделять добавление библиотек в отдельные коммиты, изменения кода приложения — в отдельные.


why?


Не вижу необходимости делать в .h файле импорт. Достаточно `@class FMDatabase`.


89976832-HPWH-fan-not-autosizing-#4693 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1650 of 1654 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.76%25-yellow.svg)


it's gc'ed once it's complete


For `hasUndirEdge`, it would probably be better to write adjacency functions (mirroring `suc` and `lsuc` by writing a new function ala `context4l` that gets all the adjacent nodes (and export these as well).


LIES


Для ещё более явного запрещения метода можно использовать атрибут `__unavailable` перед именем метода. Тогда в клиентском коде сразу будет подсвечиваться ошибка.


89976832-HPWH-fan-not-autosizing-#4693 (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


90298376_ResultsViewerUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Thank you for your reply, Now I use properties-maven-plugin with version 1.0-alpha-2 in maven, I want to read properties file and set properties to other plugin, but I cant get the properties from the file.
<plugin>
        <groupId>org.codehaus.mojo</groupId>
        <artifactId>properties-maven-plugin</artifactId>
        <version>1.0-alpha-2</version>
        <executions>
          <execution>
		    <id>read-properties</id>
            <phase>validate</phase>
            <goals>
              <goal>read-project-properties</goal>
            </goals>
            <configuration>
              <files>
                <file>src/main/resources/conf.properties</file>
              </files>
            </configuration>
          </execution>
        </executions>
      </plugin>
<plugin>
         <groupId>org.apache.tomcat.maven</groupId>
         <artifactId>tomcat7-maven-plugin</artifactId>
         <version>2.0</version>
         <configuration>
            <url>${url}</url>
            <username>${user}</username>
            <password>${password}</password>
			<update>true</update>
			<path>/UtmostTPL</path>
        </configuration>
       </plugin>
do you know why? thank you very much


Некоторые методы модифицируют базу и не закрывают её, некоторые модифицируют и закрывают. Если есть такие различия, они должны быть отражены в публичном интерфейсе и/или документации. Лучше чтобы таких отличий не было вовсе.


Cool, but why 3.2 and not 3.3 ?


Смысл очереди в том, чтобы можно было асинхронно добавлять в неё задания. В этом методе не вижу какого-либо смысла в использовании.


Can you make each of these (plate, row, and col) a separate function?

There may be some opportunity to unify parts of that, too... I see a lot of common lines!


В чём смысл этого оператора?


See if this works:
```
def qualified_count(user)
  (self.relevant_skills & user.current_skills).count
end
```
and also look into if ```self.``` is needed.


@czue, looks like the `escapejs` filter isn't actually needed for the null filter to work. Sorry 'bout that. Removed here: https://github.com/dimagi/commcare-hq/pull/5888.


90298376_ResultsViewerUpdate (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Yes


Just "the web interface". The version will depend on what they download.


`$ grunt`, to be consistent with the previous console code.


this is written just to know how to work with github


develop (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Насколько я понимаю, на самом деле дефолтным значением будет не то, которое описано как дефолтное, а просто нулевое. В случае если мы поменяем значения в перечислении, оно может стать и вовсе недопустимым.


Looks like this, plus `setUp()` below, is leftover from a test case that has been moved out.


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/1030


good point, let switch back to Object.create, as it can achieve the same.


actually I missread you. there's a :ctrl channel passed with the request to the response handler that should receive an exception, and allow you to close or whatever.


Насколько я понимаю, на самом деле дефолтным значением будет не то, которое описано как дефолтное, а просто нулевое. В случае если мы поменяем значения в перечислении, оно может стать и вовсе недопустимым. То есть следует проверить, есть ли объект по интересующему ключу, и если нет — использовать значение `xxxDefault`.


@scytacki Please refer to this commit (https://github.com/concord-consortium/rigse/commit/19959e38e8415aa7325c62a865f4a0d85874f964) for the partials.


That looks good.


Может стоило добавить для свойства атрибут `copy`, чтобы гарантировать что значение всегда будет скопировано?


saw approval from jdm
at https://github.com/glennw/servo/commit/84c5db900f5b7253efcadb0b9f1e0cc052500674


merging glennw/servo/fix-subpage-reflow = 84c5db90 into auto


glennw/servo/fix-subpage-reflow = 84c5db90 merged ok, testing candidate = b4b2c63c


5.0.1, 22


This is Hadi I got it.


fair question, I'll try an alternate implementation as a direct FrameRW class and compare / contrast tomorrow


It's confusing that you use `R`, `G`, `B` here and `r`, `g`, `b` in the following methods.  I believe the latter is more in accord with LSST naming conventions, although something like the `imageR`, `imageG`, `imageB` that you use in `makeRGB()` might be even better.


4.4.2, 19


Thanks @steveyken - I will refactor it.


good catch, will fix


yes I added TXT and DNAME records, and I try to create a clear log format to see ,and easy to process by script, ;)


Art


No realice ningun cambio de valor, solo quice hacer un commit que mostraa que hasta ahora este es el codigo que esta bien (Y))


Well, just delete all the mail, open Gitter, check unread and check for GitHub notifications. It ain't hard!


Alignment


I'm fine with that, but if I return the interface I doom myself to typecasting in `WithPodStatus`, which I'm habitually somewhat wary of. Wdyt, rock or hard place?


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/a919a140-ecd0-424a-857f-078327795bab/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/a919a140-ecd0-424a-857f-078327795bab/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/a919a140-ecd0-424a-857f-078327795bab/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/a919a140-ecd0-424a-857f-078327795bab/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/01d5953e-f91e-4b50-893c-be5e6adb58ae/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/01d5953e-f91e-4b50-893c-be5e6adb58ae/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/01d5953e-f91e-4b50-893c-be5e6adb58ae/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/01d5953e-f91e-4b50-893c-be5e6adb58ae/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/a5690885-88ad-46b5-a0e8-bd2a43b131cf/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/a5690885-88ad-46b5-a0e8-bd2a43b131cf/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/a5690885-88ad-46b5-a0e8-bd2a43b131cf/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/a5690885-88ad-46b5-a0e8-bd2a43b131cf/Chrome/elements.button.700px.baseline.png)


develop (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


90447068-BalancedExhaust-Issue4711 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1654 of 1655 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


On the device side we have to make any configuration?


This is a comment


These instance attributes are not defined for a generic `Mapping`.  Should this perhaps be calling an overridable method?


sdfsdf


@mpapo Did you *move* instead of copy the text into the icon title on purpose, or was it unintended?

See https://issues.jenkins-ci.org/browse/JENKINS-26186?jwupdated=25441&focusedCommentId=223838#comment-223838

Thanks


Note: The file uses tabs while you use spaces


Thanks, removed this fetch.


Yes, the argument is used in https://github.com/abrt/docker-abrt/blob/master/Dockerfile


1.1.3 seems to be the latest. But I will change this in a follow-up.


89976832-HPWH-fan-not-autosizing-#4693 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (638 of 638 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-4.21%25-green.svg)


This is the content for my new file.


2/2 GREAT JOB!


ViewController — совершенно точно не то место, где кто-то хотел бы видеть работу с файлами.


2/2 Good job!


Thanks! I actually haven't received anything regarding hw2, and I was
wondering would you happen to know anything about it?

On Mon, Mar 16, 2015 at 11:14 PM, Junyu Wang <notifications@github.com>
wrote:

> 2/2 GREAT JOB!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shawnshuang/sp15-hw3/commit/afc5d0a6c1a818d1e8a3307aaaa7ab16c7188cb1#commitcomment-10229279>
> .
>

--
Shawn S. Huang
B.A. Integrative Biology, Computer Science Minor
University of California, Berkeley | Class of 2015
shawnhuang1992@gmail.com


also, you should put
```head = NULL``` and ```tail = NULL```


Results provided on your One M8 device by Build Prop Editor version 1.1.8 from joeykrim in the Android Market - http://goo.gl/QKI1T

Source /system/build.prop:

# begin build properties
# autogenerated by buildinfo.sh
ro.build.id=LRX22C
ro.build.display.id=cm_m8-userdebug 5.0.1 LRX22C 66fd28a96e test-keys
ro.build.version.incremental=66fd28a96e
ro.build.version.sdk=21
ro.build.version.codename=REL
ro.build.version.all_codenames=REL
ro.build.version.release=5.0.1
ro.build.date=So 7. Dez 12:55:33 CET 2014
ro.build.date.utc=1417953333
ro.build.type=userdebug
ro.build.user=noname
ro.build.host=hacker-engine
ro.build.tags=test-keys
ro.product.brand=htc
ro.product.name=cm_m8
ro.product.board=MSM8974
# ro.product.cpu.abi and ro.product.cpu.abi2 are obsolete,
# use ro.product.cpu.abilist instead.
ro.product.cpu.abi=armeabi-v7a
ro.product.cpu.abi2=armeabi
ro.product.cpu.abilist=armeabi-v7a,armeabi
ro.product.cpu.abilist32=armeabi-v7a,armeabi
ro.product.cpu.abilist64=
ro.product.manufacturer=htc
ro.product.locale.language=en
ro.product.locale.region=US
ro.wifi.channels=
ro.board.platform=msm8974
ro.build.characteristics=default
ro.cm.device=m8
# end build properties
#
# from device/htc/m8/system.prop
#
# system.prop for m8

ro.product.model=One M8

# RIL
rild.libargs=-d /dev/smd0
rild.libpath=/vendor/lib/libril-qc-qmi-1.so
ril.subscription.types=NV,RUIM
ro.baseband.arch=msm
ro.data.large_tcp_window_size=true
ro.ril.hsdpa.category=14
ro.ril.hsupa.category=6
ro.ril.hsxpa=4
ro.ril.disable.cpc=1
ro.ril.def.agps.mode=1
ro.telephony.call_ring.multiple=false
ro.use_data_netmgrd=true
persist.data.netmgrd.qos.enable=true
persist.radio.jbims=1
persist.rild.nitz_plmn=
persist.rild.nitz_long_ons_0=
persist.rild.nitz_long_ons_1=
persist.rild.nitz_long_ons_2=
persist.rild.nitz_long_ons_3=
persist.rild.nitz_short_ons_0=
persist.rild.nitz_short_ons_1=
persist.rild.nitz_short_ons_2=
persist.rild.nitz_short_ons_3=
DEVICE_PROVISIONED=1

# QCOM
com.qc.hardware=true
persist.thermal.monitor=true
ro.qc.sdk.sensors.gestures=true
ro.vendor.extension_library=/vendor/lib/libqc-opt.so
ro.qualcomm.perf.cores_online=2

# Audio
audio.offload.buffer.size.kb=32
audio.offload.gapless.enabled=false
av.offload.enable=true
av.streaming.offload.enable=true
audio.offload.24bit.enable=1
audio.offload.pcm.enable=true
persist.audio.fluence.voicecall=true
persist.audio.fluence.speaker=true
ro.qc.sdk.audio.fluencetype=fluence
tunnel.audio.encode=true
use.voice.path.for.pcm.voip=false

# AV
media.aac_51_output_enabled=true
mm.enable.smoothstreaming=true

# GPS
persist.gps.qc_nlp_in_use=0
ro.gps.agps_provider=1
ro.qc.sdk.izat.premium_enabled=0
ro.qc.sdk.izat.service_mask=0x0

# Graphics
debug.egl.hw=1
debug.mdpcomp.logs=0
debug.sf.hw=1
dev.pm.dyn_samplingrate=1
persist.hwc.mdpcomp.enable=true
ro.opengles.version=196608
ro.sf.lcd_density=480

# Input
ro.input.noresample=1

# NFC
debug.nfc.fw_download=true
debug.nfc.fw_boot_download=false

# Recovery
ro.cwm.enable_key_repeat=true
ro.cwm.repeatable_keys=114,115

# USB
persist.sys.usb.config=mtp

# Wifi
wifi.interface=wlan0

#
# ADDITIONAL_BUILD_PROPERTIES
#
keyguard.no_require_sim=true
ro.com.android.dateformat=MM-dd-yyyy
ro.carrier=unknown
dalvik.vm.heapstartsize=16m
dalvik.vm.heapgrowthlimit=192m
dalvik.vm.heapsize=512m
dalvik.vm.heaptargetutilization=0.75
dalvik.vm.heapminfree=2m
dalvik.vm.heapmaxfree=8m
ro.hwui.texture_cache_size=72
ro.hwui.layer_cache_size=48
ro.hwui.r_buffer_cache_size=8
ro.hwui.path_cache_size=32
ro.hwui.gradient_cache_size=1
ro.hwui.drop_shadow_cache_size=6
ro.hwui.texture_cache_flushrate=0.4
ro.hwui.text_small_cache_width=1024
ro.hwui.text_small_cache_height=1024
ro.hwui.text_large_cache_width=2048
ro.hwui.text_large_cache_height=1024
ro.rommanager.developerid=cyanogenmod
ro.com.google.clientidbase=android-google
ro.url.legal=http://www.google.com/intl/%s/mobile/android/basic/phone-legal.html
ro.url.legal.android_privacy=http://www.google.com/intl/%s/mobile/android/basic/privacy.html
ro.com.android.wifi-watchlist=GoogleGuest
ro.setupwizard.enterprise_mode=1
ro.com.android.dataroaming=false
ro.build.selinux=1
persist.sys.dalvik.multithread=false
persist.sys.dun.override=0
persist.sys.root_access=1
ro.cm.version=12-20141207-UNOFFICIAL-m8
ro.cm.releasetype=UNOFFICIAL
ro.modversion=12-20141207-UNOFFICIAL-m8
ro.cmlegal.url=http://www.cyanogenmod.org/docs/privacy
persist.sys.recovery_update=false
ro.cm.display.version=12-20141207-UNOFFICIAL-m8
ro.config.notification_sound=Argon.ogg
ro.config.alarm_alert=Hassium.ogg
ro.config.ringtone=Orion.ogg
persist.sys.dalvik.vm.lib.2=libart.so
dalvik.vm.isa.arm.features=div
dalvik.vm.lockprof.threshold=500
net.bt.name=Android
dalvik.vm.stack-trace-file=/data/anr/traces.txt


develop (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Hi Yann Dupont,

could you replace my redhat mail id (ggarg@redhat.com) to my personal mail id (by using which i contribute ) garg.gaurav52@gmail.com


I think this came up before. What you have works but it's not efficient. There's a discussion board post about it.


1.5/2 seems like you forget to change the "name" attribute of user to "username"


Then you should probably not use `dirname( __FILE__ )` at all.


Please upload the whole project not a .txt file..


That's because the current lwjgl version check [enables `gl30`](https://github.com/libgdx/libgdx/blob/master/backends/gdx-backend-lwjgl/src/com/badlogic/gdx/backends/lwjgl/LwjglGraphics.java#L246-L251) even when `useGL30` is `false`. This should be [fixed](https://github.com/xoppa/libgdx/blob/2ad6b93e7b74213449516625f7aa0420bf772abd/backends/gdx-backend-lwjgl/src/com/badlogic/gdx/backends/lwjgl/LwjglGraphics.java#L301-L302) with PR #2941.


Also this is the first homework, not the secnd


Woot? That's about the first or second thing i tried... I even tried to mess around with shell_exec to change dir... Well, if chdir works pull request it, i'll find why it won't for me. Btw, i'm running windows 10 with latest wamp x64.


douban comment with one start


@fathiiiii kantoi ni hehe


It has been requested before. I do plan supporting marathons when I get back to work on gettc. But I've been busy lately. Stay tuned.


There was a conversation somewhere about security.
https://github.com/kenwheeler/slick/issues/1058


The current pull is missing to remove the setting from the DB during the upgrade stage i.e. `$module->version` should be increased and the setting should be removed during the upgrade stage via [`unset_config()`](https://github.com/moodle/moodle/blob/51fef1de359439c79fa98bd9ebec19077e66cb57/lib/moodlelib.php#L1502). An example: https://github.com/moodle/moodle/blob/42d81ef740d998e945f30ad64d1c1b9237b1d752/lib/db/upgrade.php#L223.

A quick guide about managing such things could be found in https://docs.moodle.org/dev/Adding_theme_upgrade_code.

HTH,
Matteo


Doesn't this negate the former logic?  Shouldn't it be
    if (renderTarget == null || renderTarget.__webglTexture == null) return;
So are you sure about this change in flow?


30414/Bucur E. Beniamin Eugen/Assignment 2.2/functii.c

your ```peek``` function should only return the head element. why does it have such complicated logic?


Aaaa? Apa ni? Haha
On Mar 17, 2015 2:21 PM, "Mohd Nordin Hussain" <notifications@github.com>
wrote:

> @fathiiiii <https://github.com/fathiiiii> kantoi ni hehe
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fathiiiii/MYRCurrency/commit/67dc2cedc81b9d07a19586ad0b464002576935b1#commitcomment-10229359>
> .
>


This implementation will increase the number of ROMA connections. Many connection initialization logs will appear like below.

```
I, [2015-03-17T14:44:39.473569 #8106]  INFO -- : Connected from 127.0.0.1:38194. I have 124 connections.
I, [2015-03-17T14:44:40.476903 #8106]  INFO -- : Connected from 127.0.0.1:38198. I have 125 connections.
I, [2015-03-17T14:44:41.485015 #8106]  INFO -- : Connected from 127.0.0.1:38202. I have 126 connections.
```

How about to use `get_connection` method to reuse connections?


commit ada access key pulak :p


2/2 Good Job!


30414/Ghiurau A. Alexandra Cristina/Assignment2.2/main.c

why so many variables?
is ```tail``` any different than ```rear```? or ```front``` from ```head```?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1031
success: http://build.servo.org/builders/linux2/builds/1031
success: http://build.servo.org/builders/android/builds/1065
success: http://build.servo.org/builders/mac1/builds/1247
success: http://build.servo.org/builders/mac2/builds/1247
success: http://build.servo.org/builders/gonk/builds/218


Access key? Twitter? Omg.
On Mar 17, 2015 2:23 PM, "Mohd Nordin Hussain" <notifications@github.com>
wrote:

> commit ada access key pulak :p
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fathiiiii/MYRCurrency/commit/67dc2cedc81b9d07a19586ad0b464002576935b1#commitcomment-10229384>
> .
>


You are the man!


90447068-BalancedExhaust-Issue4711 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1654 of 1655 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


2/2 Great Job!


why?


@link-er сорри за занудство), но ты можешь перенести это в файл style.css либо в отдельный файл стилей? Я не хочу менять будстраповский файл так же как переопределять будстраповские классы.


fast-forwarding master to auto = b4b2c63c


update fireball-x/dev#418


Ты всегда обязан явно отписываться от подобного рода нотификаций.


@MELERIX RE2/Biohazard 2 had expansion pack disabled because the resolution scaling causes problems with Glide64.


Or is it the Fourth project? :-)


Thanks for your fix! But you should do more adjustment work to align with my original design.
It will be helpful to avoid introducing more other issues.
For example you change doesn't support IPv6 very well because you define the socket instance address family as AddressFamily.InterNetwork (IPv4 only) explicity.

Client = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);

There must be other small isssues beside this one. But anyway, it is a great progress. Come on please, I am exiting to see somebody else can join me to develop good open source projects.


i dont think so...


Complete this function,
remove ipdb debuggin


Hey. I have tried these sources with the stock config file on my Odroid U2 and the blobs provided on builder.mdrjr.net in the u3 directory. The setup is X11-less so I chose the fbdev drivers. However, it seems there is something wrong in the kernel, see here: http://forum.odroid.com/viewtopic.php?f=9&t=10919

r4p0 with the .28 version of this kernel works fine.


OKです


again and again,thanks for the correction bro :D @falconflame


@dlbucci any idea why you removed this?


2/2 1.5 for not changing the column name from "name" to "username", +0.5 for doing extra work (show page etc)


another stupid bugs :v


Please add your name to it.


I probably accidentally removed it when I was merging conflicts.


На самом деле пользователь не может поменять имя сервиса, т.к. поле ввода нередактируемое.


I changed this function to just use Date.parse(), which won't work with the current string format. However, in my fork of the main project (https://github.com/McSwindler/home-assistant) I have changed all the datetime.now()'s to use .utcnow() and modified the string format to be ISO-8601.
I also added a couple modifications to the recorder to make sure the UTC timestamps are set/gotten correctly.
With those modifications everything seems to be working fine for me, I can switch timezones with no issues.

In my main fork, I also have some modifications to the thermostat components (mainly nest) and added a milight platform that's not completely done yet.
Feel free to pull what you want.


And the column that we do not define is id


Ah, here is the real fix.  Ideally, squash it into ef5abed5dec4d62d3304747a1ea9adb1411b5b90.


это общая беда, что до этого у гугл код, что здесь - лечится utf-8 :)


Looks like you isspelled "conala"!


Ideally squash that into former commit.


maybe this check should be moved to line 364? it is an "unlikely" situation where localhost is part of the hostlist?


Maybe we should move beyond profanity in new code?


2/2 Good Job!


Wäre es beim Installieren nicht klüger, den Server erst in der Liste anzuzeigen wenn dieser fertig installiert wurde? Bzw. frühestens wenn der Execute callback aufgerufen wird?


Okay for the client side problem. But you didn't fix the server side problem. The server crashes because we throw in the nether.


내 send가 업뎃ㄷ됐네?


ert_tests.ecl.test_fortio uses this


0.5/2 I didn't find the dog.rb in model folder and its corresponding controller, and also I didn't find any views (index, show, create)


@wonzhq thanks. you are right. looks good to me.


No this speeds up the whole processes a lot.
Scanning takes about 3 seconds a page and if we only scan 5 of the 30 days that speeds it up nicely.
Items that are for sale for longer then 5 days, i and all other traders missed or ignored anyways.

Next up is that the parseLink function makes the game client crash if i spam it. So i have to sleep between way less items then i used to. I do not know how to fix that yet,


He he this is a funny commit. :smile:


:tada: :dancer:


I think the original way is good.
In my situation, I don't want to add any html_options. I only want text. However, it cause problem like issue #87 .

Example code:
```erb
<%= link_to dashboard_path do %>
   <%= icon :tachometer, "Dashboard" %>
<% end %>
```


Support for shortened link

https://github.com/patt0/gde-app/commit/f2b0ed375642fa839bd333a2f21bc02aae525a39

Killed the quota !!!!

Patrick Martinent

On 17 March 2015 at 08:28, Patrick Martinent <patrick.martinent@gmail.com>
wrote:

> Added a line to decode the url and all that is left are the play lists
> which we will NOT support for now.
>
>
> https://github.com/patt0/gde-app/commit/4814c9ef34a2c7c378ba46f57a5163f088a71003
>
>
> Patrick Martinent
>
>
>
> On 17 March 2015 at 07:50, Patrick Martinent <patrick.martinent@gmail.com>
> wrote:
>
>> My concern was that there could be meta data with video links that were
>> created by hand, in which case your code would ignore some activity records.
>>
>> I.e. there are two videos as part of an activity, the one that is been
>> added as part of the link in the activity record and the other is in the
>> metadata, when this code executes, it will not add another meta entry for
>> the first video.
>>
>> I also am not sure how this relates to Google+ changing anything.  I will
>> conduct more experiments and list all the so-called malformed videos.
>>
>> Best
>>
>>
>> Patrick Martinent
>>
>>
>>
>> On 17 March 2015 at 04:57, Mauro Solcia <notifications@github.com> wrote:
>>
>>> As per pointed out in this comment
>>> <https://github.com/patt0/gde-app/commit/2255cd71636c5d76b02a2dc34a15c5648cdcc21f#commitcomment-10213994>
>>> it's most likely that Google + change the shared URL to one different from
>>> the one we are going to input in the meta.link.
>>>
>>> I think we could instead do something like this gist
>>> <https://gist.github.com/SmokyBob/17235b15865fdbb98c78>
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/patt0/gde-app/commit/f0512f47eca2b8cfc6eb2a78e460de0d7816b857#commitcomment-10225455>
>>> .
>>>
>>
>>
>


should I publish?


can i copy this


How is this supposed to ever get matched if the same pattern is not matched on line 60?


Argument names would be nice here.


I'm going to roll this one back because it uses *so much* CPU and makes the site_diff process significantly slower.


Sure, I will modify it


yes


Shall we use  $(nostdinccore) ?


OK, I'll change.


Следует убирать автосгенерированные шапки файлов, или хотя бы следить за их содержимым. У Новео нет копирайта на ваш код :).


What about thread safety?


Следует следить за однородностью стиля кодирования. В том числе быть последовательным и либо всегда указывать явно атрибут `strong`, либо всегда не указывать.


Yes, I'll change.


ye


Thank you for taking a look. @tchaikov


b


Hey Bryan, did you run into any trouble with 1080p video recording in your msm8660-common lollipop travels?  I'd be interested to know if you had to fix it and how you did it.  Thanks


Необходимо очищать код от неиспользуемых переопределений методов.


faster?


Сейчас мы можем только вручную обеспечивать соответствие индексов в этом методе и в методе `indexOfPasswordStrength`. Это ненадёжно и некрасиво.


0.5/2 hmm I think you might misunderstood what the question asked you to do... instead of making a user become a dog (lol), you should create a dog model, controller and views


We can't really do that this way since Google Analytics is optional in config: https://github.com/Elao/curvytron/blob/master/config.json.sample

Tracking needs to be done in the Analyser service (that perform tracking only if GA is enabled).


The method is synchronized...


Нет необходимости перезагружать всю таблицу, не так ли?


I am note sure, but I think that method is bull shit. It is because angular has built-in the same.


Не стоит ставить более одного пробела подряд где-либо кроме начала строки.


Названия переменных и значений перечисления следует писать с маленькой буквы.


Np ;) last time i switch home and other shit but now i have back time


2/2 Good Job, and nice try with flash error message!


Не лучшее название свойства, т.к. оно противоречит типу значения. Подошло бы `storageType` или что-то подобное.


Even if the g900t uses the same chip, that still depends on which technology the NFC tag uses. Mifare Classic at least does not work yet, [but already produces some debug output in logcat that might be helpful to someone who knows more about NFC than I do](http://forum.xda-developers.com/galaxy-s5/help/nfc-problems-cm12-based-mods-kltexx-t3041631) ;) Ultralight/EMV work fine.


My suggestion is to create a "getPhoneCount" method within PhoneWrapper. This would avoid redundant code we have there in 3 places. Additionally, simple caching could be made to store phoneCount inside a variable when getting it for the first time. It's something that doesn't change during runtime. This would save some CPU as methods within we get phoneCount are executed quite often.
I mean something similar we have for hasMsimSupport method at https://github.com/GravityBox/GravityBox/blob/lollipop/src/com/ceco/lollipop/gravitybox/PhoneWrapper.java#L215 (value is cached in mHasMsimSupport)


Hey thanks!!!

 Best regards, Francis Baklinski.

> On Mar 16, 2015, at 11:33 PM, Starring Forth <notifications@github.com> wrote:
>
> Or is it the Fourth project? :-)
>
> —
> Reply to this email directly or view it on GitHub.
>