41:1: I100 Imports statements are in the wrong order. import os should be before import sys
42:1: I100 Imports statements are in the wrong order. import inspect should be before import os
44:1: I100 Imports statements are in the wrong order. import cgi should be before import traceback
47:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
48:1: I100 Imports statements are in the wrong order. from operator should be before from flask
48:1: I201 Missing newline before sections or imports.
49:1: I201 Missing newline before sections or imports.
50:1: I201 Missing newline before sections or imports.
52:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
53:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_CACHED_FORMATS, CFG_BIBFORMAT_DISABLE_I18N_FOR_CACHED_FORMATS, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_SITE_LANG
53:1: F401 'CFG_BIBFORMAT_CACHED_FORMATS' imported but unused
53:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
54:6: E126 continuation line over-indented for hanging indent
59:6: E126 continuation line over-indented for hanging indent
60:1: I101 Imported names are in the wrong order. Should be create_record, record_empty, record_get_field_instances, record_get_field_value, record_get_field_values, record_xml_output
61:6: E126 continuation line over-indented for hanging indent
70:1: I101 Imported names are in the wrong order. Should be gettext_set_language, language_list_long, wash_language
70:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from .models
70:1: I201 Missing newline before sections or imports.
71:6: E126 continuation line over-indented for hanging indent
75:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_JINJA_TEMPLATE_EXTENSION, CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION, CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_OUTPUTS_PATH, CFG_BIBFORMAT_TEMPLATES_DIR, InvenioBibFormatError
75:1: I201 Missing newline before sections or imports.
75:1: F401 'CFG_BIBFORMAT_TEMPLATES_DIR' imported but unused
75:1: F401 'CFG_BIBFORMAT_OUTPUTS_PATH' imported but unused
76:6: E126 continuation line over-indented for hanging indent
82:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter.utils should be before from .config
83:6: E126 continuation line over-indented for hanging indent
84:1: I101 Imported names are in the wrong order. Should be CFG_HTML_BUFFER_ALLOWED_ATTRIBUTE_WHITELIST, CFG_HTML_BUFFER_ALLOWED_TAG_WHITELIST, HTMLWasher
85:6: E126 continuation line over-indented for hanging indent
88:1: I100 Imports statements are in the wrong order. from invenio.modules.knowledge.api should be before from invenio.utils.html
89:1: I100 Imports statements are in the wrong order. from invenio.ext.template should be before from invenio.modules.knowledge.api
90:1: I100 Imports statements are in the wrong order. from HTMLParser should be before from invenio.ext.template
90:1: I201 Missing newline before sections or imports.
91:1: I201 Missing newline before sections or imports.
99:27: E261 at least two spaces before inline comment
100:28: E111 indentation is not a multiple of four
100:28: E113 unexpected indentation
101:28: E111 indentation is not a multiple of four
101:28: E113 unexpected indentation
102:28: E111 indentation is not a multiple of four
102:28: E113 unexpected indentation
105:28: E111 indentation is not a multiple of four
105:28: E113 unexpected indentation
119:33: E225 missing whitespace around operator
210: D203 Expected 1 blank line *before* class docstring, found 0
214: D401 First line should be imperative: 'Return', not 'Returns'
214: D400 First line should end with '.', not 's'
226: D401 First line should be imperative: 'Return', not 'Returns'
251: D102 Docstring missing
290: D103 Docstring missing
296: D401 First line should be imperative: 'Format', not 'Formats'
296: D400 First line should end with '.', not 'f'
296: D205 Blank line missing between one-line summary and description
343:45: E127 continuation line over-indented for visual indent
486:27: E225 missing whitespace around operator
489:5: E303 too many blank lines (2)
507:82: E225 missing whitespace around operator
549: D103 Docstring missing
565: D401 First line should be imperative: 'Return', not 'Returns'
565: D202 No blank lines allowed *after* function docstring, found 1
565: D400 First line should end with '.', not 'g'
565: D205 Blank line missing between one-line summary and description
586:64: E261 at least two spaces before inline comment
586:65: E262 inline comment should start with '# '
593:38: E261 at least two spaces before inline comment
593:39: E262 inline comment should start with '# '
594:48: E261 at least two spaces before inline comment
594:49: E262 inline comment should start with '# '
597:24: E127 continuation line over-indented for visual indent
606: D103 Docstring missing
609: D400 First line should end with '.', not 's'
609: D200 One-line docstring should not occupy 3 lines
622: D290 White spaces around docstring should be trimmed.
622: D400 First line should end with '.', not 'a'
622: D205 Blank line missing between one-line summary and description
654:57: E126 continuation line over-indented for hanging indent
661:5: E303 too many blank lines (2)
720: D401 First line should be imperative: 'Evalute', not 'Evalutes'
740: D401 First line should be imperative: 'Analyse', not 'Analyses'
740: D202 No blank lines allowed *after* function docstring, found 1
809: D401 First line should be imperative: 'Return', not 'Returns'
809: D400 First line should end with '.', not 't'
809: D205 Blank line missing between one-line summary and description
908:13: B901 blind except: statement
939:78: E225 missing whitespace around operator
940:54: E225 missing whitespace around operator
972:47: E261 at least two spaces before inline comment
987:27: E225 missing whitespace around operator
1047: D401 First line should be imperative: 'Filter', not 'Filters'
1057: D401 First line should be imperative: 'Searche', not 'Searches'
1057: D400 First line should end with '.', not 's'
1057: D205 Blank line missing between one-line summary and description
1083:9: E303 too many blank lines (2)
1097:5: E303 too many blank lines (2)
1100:1: E302 expected 2 blank lines, found 1
1100: D401 First line should be imperative: 'Return', not 'Returns'
1114:5: F841 local variable '_' is assigned to but never used
1117:12: E127 continuation line over-indented for visual indent
1126:16: E126 continuation line over-indented for hanging indent
1148:5: B901 blind except: statement
1160: D401 First line should be imperative: 'Return', not 'Returns'
1160: D400 First line should end with '.', not 's'
1180:16: E126 continuation line over-indented for hanging indent
1187:1: E302 expected 2 blank lines, found 1
1187: D401 First line should be imperative: 'Return', not 'Returns'
1187: D400 First line should end with '.', not 'e'
1219:13: E303 too many blank lines (2)
1233: D401 First line should be imperative: 'Return', not 'Returns'
1267:16: E127 continuation line over-indented for visual indent
1277:31: E126 continuation line over-indented for hanging indent
1304:9: B901 blind except: statement
1316:17: E126 continuation line over-indented for hanging indent
1323:1: E302 expected 2 blank lines, found 1
1323: D401 First line should be imperative: 'Return', not 'Returns'
1323: D400 First line should end with '.', not 'e'
1367:1: E302 expected 2 blank lines, found 1
1367: D401 First line should be imperative: 'Return', not 'Returns'
1367: D202 No blank lines allowed *after* function docstring, found 1
1494:1: E302 expected 2 blank lines, found 1
1494: D401 First line should be imperative: 'Return', not 'Returns'
1494: D202 No blank lines allowed *after* function docstring, found 1
1521:17: E225 missing whitespace around operator
1589:1: E302 expected 2 blank lines, found 1
1589: D401 First line should be imperative: 'Return', not 'Returns'
1589: D400 First line should end with '.', not 't'
1633:28: E261 at least two spaces before inline comment
1633:29: E262 inline comment should start with '# '
1681:5: B901 blind except: statement
1787:1: E302 expected 2 blank lines, found 1
1787: D401 First line should be imperative: 'Return', not 'Returns'
1787: D202 No blank lines allowed *after* function docstring, found 1
1787: D400 First line should end with '.', not '}'
1804:56: E225 missing whitespace around operator
1828:1: E302 expected 2 blank lines, found 1
1828: D401 First line should be imperative: 'Return', not 'Returns'
1828: D400 First line should end with '.', not 'e'
1857:26: F821 undefined name 'exc'
1859:22: F821 undefined name 'exc'
1862:1: E302 expected 2 blank lines, found 1
1862: D401 First line should be imperative: 'Return', not 'Returns'
1895:1: E302 expected 2 blank lines, found 1
1895: D401 First line should be imperative: 'Return', not 'Returns'
1939:1: E302 expected 2 blank lines, found 1
1950:1: E302 expected 2 blank lines, found 1
1950: D203 Expected 1 blank line *before* class docstring, found 0
1950: D204 Expected 1 blank line *after* class docstring, found 0
1950: D400 First line should end with '.', not 'n'
1950: D205 Blank line missing between one-line summary and description
1981:15: E261 at least two spaces before inline comment
1983: D401 First line should be imperative: 'Create', not 'Creates'
2015:31: E261 at least two spaces before inline comment
2039: D401 First line should be imperative: 'Return', not 'Returns'
2039: D400 First line should end with '.', not 'e'
2054: D401 First line should be imperative: 'Return', not 'Returns'
2054: D400 First line should end with '.', not 'd'
2077: D401 First line should be imperative: 'Return', not 'Returns'
2077: D400 First line should end with '.', not 'e'
2077: D205 Blank line missing between one-line summary and description
2115: D401 First line should be imperative: 'Return', not 'Returns'
2115: D202 No blank lines allowed *after* method docstring, found 1
2224:36: E127 continuation line over-indented for visual indent
2227: D401 First line should be imperative: 'Return', not 'Returns'
2284:33: E127 continuation line over-indented for visual indent
2295:60: E251 unexpected spaces around keyword / parameter equals
2297:54: E251 unexpected spaces around keyword / parameter equals
2307:37: E127 continuation line over-indented for visual indent
2318:64: E251 unexpected spaces around keyword / parameter equals
2320:58: E251 unexpected spaces around keyword / parameter equals


We'll use this app or cmupdater? If we keep this one i rather we use it as pa does. Let's keep in launcher and not inside settings.


4: L102 copyright year is outdated, expected 2015 but got 2014
31:1: F401 'zlib' imported but unused
33:1: I101 Imported names are in the wrong order. Should be CFG_INSPIRE_SITE, CFG_OAI_ID_FIELD, CFG_WEBSEARCH_FULLTEXT_SNIPPETS, CFG_WEBSEARCH_FULLTEXT_SNIPPETS_CHARS, CFG_WEBSEARCH_FULLTEXT_SNIPPETS_GENERATOR
33:1: F401 'CFG_OAI_ID_FIELD' imported but unused
34:6: E126 continuation line over-indented for hanging indent
41:1: I100 Imports statements are in the wrong order. from invenio.utils.text should be before from invenio.utils.url
41:1: F401 'encode_for_xml' imported but unused
42:1: I100 Imports statements are in the wrong order. from invenio.utils.shell should be before from invenio.utils.text
43:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.utils.shell
43:1: F401 'get_fieldvalues' imported but unused
45:1: E302 expected 2 blank lines, found 1
45: D202 No blank lines allowed *after* function docstring, found 1
45: D400 First line should end with '.', not "'"
60: D290 White spaces around docstring should be trimmed.
60: D400 First line should end with '.', not 'g'
67:1: E302 expected 2 blank lines, found 1
67: D401 First line should be imperative: 'Return', not 'Returns'
67: D202 No blank lines allowed *after* function docstring, found 1
67: D400 First line should end with '.', not 's'
67: D205 Blank line missing between one-line summary and description
101:1: E302 expected 2 blank lines, found 1
101: D401 First line should be imperative: 'Return', not 'Returns'
101: D202 No blank lines allowed *after* function docstring, found 1
101: D400 First line should end with '.', not 'n'
101: D205 Blank line missing between one-line summary and description
112: D401 First line should be imperative: 'Grade', not 'Grades'
130: D401 First line should be imperative: 'Grade', not 'Grades'
143:47: E225 missing whitespace around operator
165: D202 No blank lines allowed *after* function docstring, found 1
165: D400 First line should end with '.', not ':'
165: D205 Blank line missing between one-line summary and description
212:29: E261 at least two spaces before inline comment
213:31: E261 at least two spaces before inline comment
214:31: E261 at least two spaces before inline comment
215:31: E261 at least two spaces before inline comment
218:24: E261 at least two spaces before inline comment
220:26: E261 at least two spaces before inline comment
223:22: E261 at least two spaces before inline comment
227:22: E261 at least two spaces before inline comment
232:26: E261 at least two spaces before inline comment
234:22: E261 at least two spaces before inline comment
238:22: E261 at least two spaces before inline comment
244:1: E302 expected 2 blank lines, found 1
244: D401 First line should be imperative: 'Return', not 'Returns'
244: D400 First line should end with '.', not 'd'
244: D205 Blank line missing between one-line summary and description
313:1: E302 expected 2 blank lines, found 1
313: D400 First line should end with '.', not 'e'
313: D205 Blank line missing between one-line summary and description
322: D209 Put multi-line docstring closing quotes on separate line
322: D400 First line should end with '.', not 'r'
322: D205 Blank line missing between one-line summary and description
359: D400 First line should end with '.', not 'd'
359: D300 Expected """-quotes, got '-quotes
365: D400 First line should end with '.', not 'd'
365: D300 Expected """-quotes, got '-quotes
385:1: E302 expected 2 blank lines, found 1
385: D400 First line should end with '.', not "'"
385: D205 Blank line missing between one-line summary and description
413:18: E261 at least two spaces before inline comment
445:1: E302 expected 2 blank lines, found 1
445: D400 First line should end with '.', not 't'
445: D205 Blank line missing between one-line summary and description
494: D202 No blank lines allowed *after* function docstring, found 1
494: D205 Blank line missing between one-line summary and description
518: D400 First line should end with '.', not 'r'
518: D205 Blank line missing between one-line summary and description
534: D290 White spaces around docstring should be trimmed.
534: D400 First line should end with '.', not ':'
534: D205 Blank line missing between one-line summary and description
534: D208 Docstring is over-indented
544:15: E261 at least two spaces before inline comment
545:16: E261 at least two spaces before inline comment
566:55: E261 at least two spaces before inline comment
569:95: E261 at least two spaces before inline comment
578: D290 White spaces around docstring should be trimmed.
578: D400 First line should end with '.', not 's'
578: D205 Blank line missing between one-line summary and description
578: D208 Docstring is over-indented
587:52: E225 missing whitespace around operator
594:36: E225 missing whitespace around operator


25:1: I201 Missing newline before sections or imports.
27:1: I100 Imports statements are in the wrong order. from flask should be before import yaml
28:1: I201 Missing newline before sections or imports.
33: D103 Docstring missing


This feels like it would be better described as `short_name`. Let the code that produces the markup for the metadata work out whether it wants to use the full name or the short name.


its identical, but it properly takes NA into account, which always makes the plotrix version to fail. One could submit this to plotrix as well at some point. But in this way we also do not depend on plotrix.


No problem !

Could you have a look at https://github.com/jmxtrans/jmxtrans2/pull/38 and tell me if it looks good enough to you for merging ?


Done


"Issued" is a good short name, but it makes me wonder whether the full name should be a bit… fuller. "Date issued", perhaps?


Shouldn't this copy method be in the very class we are copying? Just like we have copyHelper in Order.java


@jcerjak I'm sorry that p.a.robotframework has not yet CI for Plone 5. Yet, would you have a good idea, how to make this work for both Plone 4 and Plone 5? (E.g. define HAS_PLONE_5 somewhere and do conditional imports etc on base of that.)


Reviewed and approved.


As with the `metadata` key, I think this would be better as `show_summaries` or something else that indicates it's about whether to show the summaries for results on the finder page.


I just went with what's [here](https://github.com/alphagov/finder-frontend/blob/master/app/models/medical_safety_alert.rb#L21).


嗯 还是加了


thanx, i think the other can be removed. will try that later.


Hi, I wanted to use this locally before it gets pulled, but pyleus keeps downloading the "real" version to its virtualenv. So how can I use your fixes if I build my own pyleus package? Thank you in advance.


Its right, thanks perfect.


I was not aware of that function. My PHP craft is very limited.
Anyway, it is still not enough, I am not able to break a build :-(


So this removes the generated sources from the compile-sources, right? Looks weird.
I think the output of the annotation processor is picked up automatically by javac, or isn't it?


Also our game has sounds now^)


What does Are you sure mean? :)


It looks weird indeed. But is part of the [`gradle-apt-plugin` plugin from Jimdo](https://github.com/Jimdo/gradle-apt-plugin/blob/master/src/main/groovy/com/jimdo/gradle/AptPlugin.groovy#L31). Don't know if really needed actually.


But uPy doesn't strictly conform to python3 with unicode disabled :)

Background to these changes: trying to get uPy compiling cleanly on 16-bit dsPIC33 with unicode disabled.

If unicode is disabled, what should `chr(0x100)` do?  In this case mp_obj_new_str will only ever use the lower 8-bits to make a 1-byte string, so I thought it best to give an error that the char cannot be represented correctly.  Otherwise it succeeds silently but incorrectly, and leads to nasty bugs.


A


das return(ranking) sollte noch optional sein - füge ich hinzu


Nicely put. That's what I was trying, but didn't managed/dared, to say. ;) While in this 'doc mode', perhaps a touchup on [this one](https://github.com/jeffreykegler/Marpa--R2/pull/232)?


PR is merged!


I think this is not needed, as the sources are picked up by the compiler anyway. We just need to make sure sources are known to AS, which is done in line 31ff.


Ok, got it.

PSR-2 says we must not have space after `if (`.
But you can align `||` inside conditions.


I came up with this chunk, but I still would prefer the original solution, as it looks cleaner.  What do you think?

```diff
diff --git a/src/libostree/ostree-repo-commit.c b/src/libostree/ostree-repo-commit.c
index 266785d..a59e405 100644
--- a/src/libostree/ostree-repo-commit.c
+++ b/src/libostree/ostree-repo-commit.c
@@ -1057,11 +1057,17 @@ rename_pending_loose_objects (OstreeRepo        *self,
       if (strncmp (out_dent->d_name, "tmpobject-", 10) == 0)
         {
           char loose_path[_OSTREE_LOOSE_PATH_MAX];
-          gs_free gchar *checksum = NULL;
+          char filename[_OSTREE_LOOSE_PATH_MAX];
+          gchar *checksum = NULL;
           OstreeObjectType type;
-          ostree_object_from_string (out_dent->d_name + 10,
-                                     &checksum,
-                                     &type);
+          char *dot;
+
+          g_strlcpy (filename, out_dent->d_name + 10, _OSTREE_LOOSE_PATH_MAX);
+          dot = strrchr (filename, '.');
+          g_assert (dot != NULL);
+          *dot = '\0';
+          checksum = filename;
+          type = ostree_object_type_from_string (dot + 1);
```


Space?


change to lazzerzz


:D Seems like something exciting is keeping you up :P


Great idea!


I can appreciate that grabbing this value via .get() was to avoid having more l10n cases, but usually that means the localized text will not be localized correctly in other languages, if they prefer a different separator and the language change is done while the app is running.

I suppose the hope is that it would only affect some nodes, since a scroll would likely cause new calls to this function then getting the right localized value? Or that all screen readers in all locales use comma space to insert a pause in the playback?

In general, for this pull request, I would like to avoid doing this extra work if we could avoid it somehow. Or parameterize it/use some HTML only visible to the screen reader for this case? Or is there something we can do in the existing HTML for an item, if we structure it differently, then could we avoid this?

This info is set on the items in the vscroll component for email, and the faster we can do the work and limit DOM changes, the better it is for not becoming the long pole for scroll performance.


Later commit fiddles with the local value of this so it will need to stay conditional until we use an 18 unless we provide more than a single version downgrade - but that doesn't seem a likely requirement.


thinking about it, this should probably through. the tests always add the lib file to the list of files, so as we do the resolution we find it, and we never need to ask the host for the library. so i believe this is never called.


OK so save the new stuff.


We are currently just a basic mutex as a lock on the creation of this singleton instead of something from tbb.  I think it should be sufficient unless someone else knows something better.


_Kill... Kill..._


please tell me, I have to leave it?
blank?


i think side-effects in an assert is a bad idea...


You call this function as `getProductImagesToHtml(Mage::registry('product')->getId())` in  `app/code/community/MVentory/Productivity/Helper/Data.php` file.

So, you have already loaded product model in `Mage::registry('product')`, but then you take its ID and load product model in `_getProductMediaGallery()` method.

Again, it's not efficient and broken. Can you please elaborate on this decision?
Can you give answers to my questions, but not silently fix them?


That came from the `raw_subscriptions` method as seen in the current versions of `Event/XPostFailure.pm` and `Event/UserMessageRecvd.pm` which I copied verbatim into the other events that were doing the same thing (before de-duplicating the identical methods in the subsequent commit).


D'oh, that particular diff hunk confused me ;-).


Events can already be attached. "enables " is the wrong world. This is more of a convenience. "When true, marker click events are automatically registered."


I did more about ContractNet, but got stacked into several not known parts
(as RBF). Tomorrow I will finish a simple ContractNet with dani's help. :D

2015-01-30 0:59 GMT+01:00 philoliver <notifications@github.com>:

> Aha. Strange... I'll wait. We can check it tomorrow on the other machines
> too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dgl3/IMAS/commit/bc4179b85bb0106ea1ed6a9381d65c2bd1b17300#commitcomment-9509460>
> .
>


QElapsed() timer is a better choice for operations timing operations than QTime() it tries hard to be monotonic on most systems (not affected by date/time changes).


This has nothing to do with the test, because the builds themselves timed out.


Do you prefer multilining all of these even when line length isn't an issue? I think of this every time I see it.


This fails when there are no missing values.
This segment should be changed to:
```
	all_days <- sort(unique(unlist(lapply(names(l), function(x) {
		names(l[[x]])
	}))))
````


Don't add it to your pt-BR strings.


not == ?


issue-4700-idd-file-directory (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1595 of 1595 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)


So, I think you don't need `type` parameter anymore.
Can you remove it and clean the code further?


originally thought it would be better from a output screen spamming if had the choice to switch on/off timing messages individually with a debugger whilst testing but got frustrated remembering to turn them on each time I wanted them to show - later commit should have nuked all the instances of this (shared) static value.


je pense que c'est inutile de mettre les chaines comme ça (qui finissent par ```/>``` ou ```"></string>```  , car des fois ils ajoutent du texte, et donc si on a celle la, qu'ont ne fais pas la modification à temps et bien à la place ca affichera un espace blanc


Is there a situation when new_apps wouldn't' be ready? (might be worth a comment)


An explanatory comment about when something should be added to `local_models` would be helpful.


"default", je me rappel sur la V5 ce "reglage" peut changer selon les modeles d appareil


we don't care that the have value is always empty?  i guess it only matters for the optimized update via clone?


Got reformatted by Qt - not sure what our style is for switches(){}.  And warning is just that - to protect against odd situations.


See here:
http://hydra.nixos.org/build/19045023
http://hydra.nixos.org/build/19044973
http://hydra.nixos.org/build/19044968
http://hydra.nixos.org/build/19045019


Fixed.

----- Original Message -----
> When using `openshift cli`, what happens?  Does it still have the same
> template?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/fabianofranz/origin/commit/a73a11c749c103b6e1b36e248ac9d8552742055a#commitcomment-9508348


:yum:


if this is return by value does the const do anything?


r+


What about using skill hashcode method?


Fixed in 90761066b236, ty


We should drop those urls with defaults={'type_name': None})


What retail bug? Could you elaborate on this?


backport?


OK, having already eliminated calls to setExit where nothing IS changed update old exit destination room.


POPCNT is sloooow.


NOW change the exit target room Id.


I don't think the templates for openhsift and cli will be the same.  I think this is a gap in Cobra, that you may not want the root command's help usage when you're running as a subcommand.

----- Original Message -----
> Hrm, good catch. `openshift cli` does, but anything else (start, etc)
> doesn't. I guess I need to move the templates to 'openshift' and make it
> used by everything, including the infra commands.
>
> ----- Original Message -----
> > When using `openshift cli`, what happens?  Does it still have the same
> > template?
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/fabianofranz/origin/commit/a73a11c749c103b6e1b36e248ac9d8552742055a#commitcomment-9508348
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/fabianofranz/origin/commit/a73a11c749c103b6e1b36e248ac9d8552742055a#commitcomment-9509451


PopCnt is 10x slower.


We should probably consider stats per task type.


looks good!


How do you find that some theme changes background of element with `productivity-tasks` ID?
Can you please provide more info?


I think this should actually be `createElement('this', null)` – sorry for the confusing comments. Your `this.foo` test is good though.


"Given _a_ settings object"


Smells like Java!


Simpler to write code that works on a pointer to the exit affected rather than copy it for each of the 12 possible exit directions.  There may well be other places where this is a valid choice.


>I believe I've sent the standard IRC commands correctly, but you're saying you've at least seen feedback in the console showing /NAMES (and the resulting list) in the chat? I don't seem to get that in mine, even with debug on.

I haven't tried this with the bot in particular. I tried it from mIRC while exploring some of the conditions Twitch has put on their IRC server.

A good example is sending the "/list" command, which should list all channels available on the server, along with counts. The list eventually will be received by mIRC, but it can take 1-5+ minutes. I believe this is the reason why it appears to not work, the request just comes in much later.

>Also, not sure if you're interested, but I should have !songrequest sending to a Spotify playlist using subprocess call() by tomorrow (most of that code is excluded from the repo). I'll include everything once I have it functioning.

I've been aiming to create a framework for others to base code off of. I'd be happy to include whatever commands/features are generic enough that others can use them.


default_scope { order("rank DESC") }


les valeurs default peuvent changer selon les téléphones donc il faut pas les mettres en "dur" en plus c'est affiché nul part inutile d'ajouter


Apparently this was removed in ec796ec2299d03ddfb821e97047c0329f11ab962


別ブランチとか別のリポジトリにしたほうが良かったのでは…？


No need I think. This flag doesn't add any value, just complicates things.

Task model should have flag closed, which is defaulted to False (and has index?).
And next endpoint should consider this value when searching for a new tasks.
Then save answer routine of plugin will be able to toggle that flag if needed.


And update the new room to have an entrance to this room.


@bbodenmiller Should insert a comma after 'remove it' and I think 'make' should be 'may' in: "...information you may remove it however it make affect our ability to assist you"


Good catch. I don't see any rule to change this behaviour. Our style guide also does not strongly specify on the spacing here beside of suggestion to use literals instead of `valueForKey`, but still it's a violation.

Can we agree to hold on on this and I'll see if it's possible to add this rule to the formatter?


s/an empty list of {{Documents}}s.//?


Not sure. Seems like if someone is using `<this />` then they want `this` and not `"this"`.


What's about `$(WAF) clean`?


`<li>` tag doesn't have `src` and `type` attributes.
Can you please rework the code?


Sorry? How boolean flag works with redundancy?


oops j'aurais du les laisser celles là


Is it necessary?


pas traduite ?


The case for an exit deletion (to is -1)


Did you actually try building 32-bit like this? As far as I know, you can't use gcc's int128 on 32-bit so they should probably still use real GMP library.


Looking for some stats? I have some sort of that!
https://github.com/mrgambal/vulyk/blob/master/vulyk/models/stats.py


Could happen - depends on other code.


Can you please elaborate on this changes? Why do you need it?


Don't put this logic here. Task should decide. The most you can add here is a call to a METHOD, that will return true or false.

During call I gave you an example when task might be closed even if not a single record repeats another one.


backport?


contador


Shouldn't happen but I think we should note if it does!


You deserve it! No complains from you anymore!


Yeah, I hear you. @sebmarkbage?


The `@brand-primary` could have been simply replaced with the default value, but I wanted the source of the new color value to be left there for context. The darker primary blue increases accessibility across the board; the link change was merely the focus and impetus for this change.


ehm, my bad...(


seems like u are make small typo

0x0B32 2866 SMSG Client 0 BFMGr (NEW 18764)

0x098B 2443 SMSG Client 0 BFMgrQueueInvite


"It's a hard knock life" ORIGINAL Annie 1982: http://youtu.be/-0bOH8ABpco


Hmm, okay, that might work, but here we should return False, I think. And move that line to a helper method that AbstractTaskType descendant can call for trivial cases.

However I'm keen to have two values for "redundancy". For example, consider this:
We want to close a task if we got 3 identical answers, or 5 answers in total, so we won't be giving this task to user over and over again, until we reach 3 identical answers


Using a QSet for this and mSpecialEntrances avoids issues of duplicate items...


Is it just one routine in particular that fails (pow)? If so, why not just disable that one.


:hand:


This is handled by the bloom automatically. But I'm just providing an off set options for a some control over it.


NO WAY! Task is just a DAO. It mustn't know anything except itself.
Task-class should handle representation only.

a) You could override this method in your own *TaskType, but not in *Task.
b) we should create another class whose responsibility would ba reports handling.


definitely more readable to me :+1:


better describes what's actually happening in that callback :+1:


I generally prefer that code is deleted rather than commented out. It just ends up leaving clutter around.


The setEntrance(QPair< <key>, quint64 >) where <key> is quint8 (unsigned byte) or QString are the only {re}setEntrance() methods that need to be public.


Yes. I was worrying about the label appearing before the caption :smile:


Ok, read my comment above as TaskType should decide, but not AbstractTaskType


Please, avoid these wildcard imports, explicit is better :snake:


Make sure @brianthomas gets this change in the proofs. Elsevier will already be working on the proofs.


both methods work actually :P


:snake::snake::snake:


HVACSizingSim_CoincidentPlant (EnergyArchmage) - 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-68.91%25-red.svg)


Got ya. Here comes one more problem - we just have to pray that MongoDB could properly compare relatively complex JS-object.
:pray:


Should have been a QMultiHash<int,int> or something else to handle the case of exits from a room that has ones that are both normal and special to the same destination...


thanks for making the change. it would be cool to add a link to the github repo from the spec (i looked for one and more and more w3c specs have it), so that people can fix trivial stuff like this.


See https://github.com/xebialabs-community/xlr-xlr-plugin/pull/3. My suggestion would be, once that is merged, to remove this repo..?


We can compare ourselves. In python. As we are only interested in a very
limited subset of objects: answers to current task.

On Fri, Jan 30, 2015 at 2:32 AM, Dmitry Gambal <notifications@github.com>
wrote:

> Got ya. Here comes one more problem - we just have to pray that MongoDB
> could properly compare relatively complex JS-object.
> [image: :pray:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrgambal/vulyk/commit/9f9bceeb6150fa87c6cfb830eb71377c11b342da#commitcomment-9509826>
> .
>


It's weird that you're calling a function called "_navigationHandler" on click.   A click is for selecting and closing the dropdown, not for navigation.

Why not put the closeDropDown() call inside the click handler?

Also, I don't see where the code is for closing the drop down based on enter key, but presumably that should follow the same code path as the click handler.


Bah, it's only 1:37am ^^


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1572 of 1610 tests passed)

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


Sounds good to me.


86121908_copy_schedule (macumber) - x86_64-MacOS-10.9-clang: Build Failed

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


By only storing the rooms that have exits that lead to this room it would have to be called every time an exit was changed and we'd still have to iterate through all the exits of the rooms listed at room deletion time.


The handler of 0x0B32 calls `FrameScript_SignalEvent(st7_0, a2, a3, BATTLEFIELD_MGR_QUEUE_INVITE, (int)"%d%b%s", *(_WORD *)(a1 + 16))`


I'm concerned that this might change behavior, in the sense of -- doing something if there's no etypeid, and doing something else if there's an etypeid, etc.


This link should be http://www.w3.org/TR/html5/browsers.html#ancestor-browsing-context or https://html.spec.whatwg.org/multipage/browsers.html#ancestor-browsing-context.


Ah, gotcha. Yeah this looks fine then


Medium: use auto incremental numbers


Have you actually benchmarked this? I remember this being beneficial even on ARM946 for whatever reason, so I've left it on for all ARMs intentionally.


I don't like this one.


nitpick: this is short-circuit logic (like above with IsActive). So just return here and you don't need the big `else`.


update_sqlite_coverage (mbadams5) - 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)


Whilst it might be data structurally possible to have an exit with no name - how could you use it? Now give feedback about the issue - return false! :speak_no_evil:


`There is no IApiResponseMetadataProvider to generate the content type user asked for`

Close... I'd say something like `We need to check if the action can generate the content type the user asked for. Otherwise...`


Medium: count(1) is better in performance


Ah, yes. Oops. The original text talked about the something team, so "they" was referring to that team. I simplified the text to just say "we", which broken the meaning of "they". Thanks for seeing it!


Clean up docs here


`format` -> `Format`


As for normal exits, if the details haven't changed we should skip the rest - successfully.


So update the old destination room about losing an entrance.


oh shit... seems like i had make small typo on dumping


private


Is this still needed?


Set the new destination room.


And tell that room about the new (special) entrance.


But they're so nice; embrace the space.
Don't cram your code up in my face.


How do I save my changes to Git on the web without spawning these tests?

On 2015-01-29 17:06, Tik-Tok wrote:
> 68304232-1e)-Move-parent-object-sizing-calcs-to-a-common-routine
> (rraustad) -
> x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests
> Failed
>
>  [1] [1]
>
> --
> Reply to this email directly or view it on GitHub [2].
>
> 	*
>
> Links:
> ------
> [1]
> https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-ff51e189b5e6d54eef4f772194e64bcffd30cd6e-x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug.html
> [2]
> https://github.com/NREL/EnergyPlus/commit/ff51e189b5e6d54eef4f772194e64bcffd30cd6e#commitcomment-9507851


filepath.Join?


High: use a singleton to wrap the connection and another class to deal with queries, etc. That singleton (or even connection pool) is just responsible to provide connections.

To deal with tables, use DAO pattern. Please fix it just for one entity and fix the auto incremental value in that entity/table.


Also need to update http://www.w3.org/2001/sw/wiki/JSON_LD_Errata


Should this be "Let _origin_ be ___ancestor___'s origin"?


no


As we no longer use this method and it would have to be rewritten to accommodate the entrance data changes we should remove it.


Would simple `$this->templateId = $className` work as well?


Does .io stand for British Indian Ocean Territory?


We don't need to have read the roomID in advance so do it here with everything else.


Should be marked as BC break.


I don't get the idea of this change. Can you please explain?


Would you be interested in getting your stuff merged into the upstream master, or https://github.com/cyaninc/git-fat (lots of ongoing development there too)?


`Model::toArray` seems to be the wrong thing to use to present a model. We want something that represents the actual value of the object and its properties, not a ready-to-be-serialized nested associative array. Is there something better?


I would add a < br / > either before or after < p >, just to separate the image from the buttons. I think it looks better.


Why not just `instance_map = ctx.instance_map = ctx.my_resolve(conf_module)`?


Would this stuff be useful for a wider audience?  eg by merging into the main repo, or https://github.com/cyaninc/git-fat (lots of ongoing development there too)?


Published on Sonatype. CLI assemblies are up. Waiting for sync to Maven to launch my new script to download and install all the ScalaDocs automatically.


Looks great!!
You will probably want `:source-map-timestamp true` in order to get up to date source maps as you live reload.
https://github.com/clojure/clojurescript/wiki/Compiler-Options#source-map-timestamp

Giving the build an `:id "dev"` will be helpful to figwheel.

When you move to the next version of figwheel the following may be helpful:

Also this function checks the config values:
https://github.com/bhauman/lein-figwheel/blob/master/sidecar/src/figwheel_sidecar/auto_builder.clj#L69
(fig-auto/autobuild-ids (assoc config :build-ids ["dev"]))

You may also want to look at the built-in repl (it's not nrepl enabled):
https://github.com/bhauman/lein-figwheel/blob/master/sidecar/src/figwheel_sidecar/auto_builder.clj#L201

Hope this helps!


or do you just want the PRs first?   Also, should i use the original bug number in these PRs as well?


(minor) I used to use `...` for that, like in `[binding(s)...]`


Huh? I don't want a table {'key'=value}


can you share your lists?


fefef


wefwef


Are you sure `old` is actually defined here? I got bitten a bunch of time with Crystal, and had to do the following instead:

```crystal
def method
  old = working_directory
  begin
    ...
  ensure
    chdir(old)
  end
end


r=jdm


this is not the actual secret to this account. It's just a properly-formatted secret (for a wallet that does not exist)


Yeah, would be a bit more elegant codewise but slower


Thank you! Always capitalize 'Rust' the language. Pet peeve.


mmmmmm


The test passes, so I would expect so.


Still missing license header in
  lang/cs/Tests/ReefTests/ReefTests.csproj
  lang/cs/Tests/TangTests/TangTests.csproj
  lang/cs/Tests/WakeTests/WakeTests.csproj


Uhhhh lol


This doesn't look right


68304232-1e)-Move-parent-object-sizing-calcs-to-a-common-routine (rraustad) - i386-Windows-7-VisualStudio-12: OK (1587 of 1595 tests passed)

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


This was the bug that caused the crash in the Web server (due to the bug fixed in 0882cea1b7f2b10f6aee56c2575621274e1e07e7).  The key was being signed with itself rather than with the signing key.  Simple fix.


I have no idea yet, still need to test it.


LOL


Oh look. I can comment on this file.


just in pm ( mail or smthng like this )


add mesh file ref. #4607


Also need to specify DH key pushes as "subkey" pushes since DH keys can't be sibkeys


Thank you for your attention to details, @H4tch!

We are in the process of switching from SimPL-2.0 to Apache License 2.0.  Would you be so kind to read the thread at http://discuss.gohugo.io/t/switching-to-apache-2-license/173 and give your permission for your contribution to fall under the new license?  Many thanks for your help!


I see how this deref stuff is annoying. Otherwise get this into the pr for one last look, but looks good to me.


Yup, see the next commit :)


Not meaningfully: it still only iterates through the changed set.

We could write a more high-performance merge function by removing the runtime type checking (and with it the support for nested tables, which we don't care about here).

In general, though, the difference between the two is very unlikely to be noticeable. At this point, we're so utterly devoid of sanity in our codebase I tend to err on the side of elegance. The performance kinda just happens by accident as you mop the blood off the walls.


So the code will already process the new format but the value has not (yet) gone to the value needed for this to be active.


@jczhang @drayson92 r


i see it


@shahata There was one internal test that errored on this line because annotatedFunctions was undefined. The cause was that the application created a separate injector to get one-off access to ngResource in a base class that was just used as a base class to other classes. We worked around it by not needing to create the injector at all, but I wonder if there's a legitimate bug here.


this isn't from fusion rom.. it's from Exodus/vanir.  If you're going to fudge the authorship like this you should at least give credit to the correct ROM or link the original commit.


me or @lanthaler or either of us?


This is needed! Those are different actions.


This method already iterates through every exit on every room so it is the ideal place to introduce the code needed to determine the room entrances - the existing code base could have had code here to do the same thing but it didn't.  It could also have had code that worked when each TRoom instance was instantiated but that would have been hampered by not being able to validate exits to room that had not yet been read (the room not the exit!)


I had no clue it was from Exdus/vanir sorry about that


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (613 of 614 tests passed)

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


You mean like it it should be a ternary?


:clap:


`-pr` or `-rp`? decide already!! ;)


What's 'lst' for?? Looks like it's just declared and not used


cheers to @patrickxb, he deserves the glory


I'm not that stressed about it bro but this is the 3rd rom i've found it in without even a mention (licenses changed and everything) or a thanks.  if it's not that difficult to do i'd at least appreciate removing the fusion ROM part since that guy was basically ripping off everyone to the point where he had to delete his frameworks source entirely lol


MountaineeringExpedition is an iOS app. And it was developed by Titaium(Alloy) framework.

App link:
https://itunes.apple.com/kr/app/deungsan-wonjeongdae/id954192848?mt=8


81605912-Multispeed-Heating-Defrost-Bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1093 of 1098 tests passed)

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


> This doesn't look right

can you explain?


I think this needs to be reworded to be more clear. "The parallel collection" needs context.


r=mbrubeck


87220608_arrowsBackwards (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2179 of 2190 tests passed)

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


This seems to assume the reader knows that the default cache size is 10. Do you mention that anywhere? If not, it should be added.


I was attempting to make an SOS signal but ran into some problems along the way; so we'll call this one a work in progress.


He must mean the ternary or the bad spacing lol.


Sure when I get home I'll rebase the 2 branches and fix the commits related to this.


I think it is misleading to say that you define  your dataset with the descriptor builder. Building a descriptor is an intermediate step.


update_sqlite_coverage (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1572 of 1610 tests passed)

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


on 이 뭐에요?


Appreciated. You might wanna diff with my source too.. I've made some bug
fixes and this looks like the vanir version which misses them.
Github.com/vanirexodus
On Jan 29, 2015 8:26 PM, "Sean Hoyt" <notifications@github.com> wrote:

> Sure when I get home I'll rebase the 2 branches and fix the commits
> related to this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LiquidSmooth/android_frameworks_base/commit/7db69b8f09216586fd709b7d0ec4cdc8d062f468#commitcomment-9510343>
> .
>


Perhaps he wanted:

```cpp
void InstrumentTrackView::muteChanged()
{
     m_activityIndicator->setActiveColor( QApplication::palette().color( QPalette::Active,
         model()->m_mutedModel.value() ? QPalette::Highlight : QPalette::BrightText
     );

}
```


Cool I will thanks for pointing this out


@Quintus / @Luiji - Using release-2.0.0, if you go to load a level, only levels with the extension *.tsclvl are displayed.

I think it would make sense to continue displaying levels with *.smclvl in the level selector.  Otherwise many players will assume TSC is not reverse compatible with SMC levels, which would go against the reverse compatibility we've been maintaining for TSC 2.0.

We could still convert the level to a .tsclvl after reading it and, in version 3.0, eventually remove the logic to display .smclvl's at all.


Synced on Maven. ScalaDoc published.
Good night ^^


Demo is not packaged with resources, images will not show without resources.


What?


String search in a directory tree


I didn't feel like copying the code from setExit( roomId, exitDirection) for each set{exitDirecition}( roomId ) and I couldn't see any benefit from retaining them....!


Wow, this is brilliant, and it works correctly!

The only possible change would be to restore line 369, because there needs to be an error check so that pairs with one (or both) species missing are excluded. This error check doesn't necessarily have to occur inside the C-score function, but it will need to occur inside the loop that creates null matrices because some algorithms can generate matrices with empty rows (and a user could possibly supply an incorrectly formatted matrix with empty rows). Ted, we should discuss this kind of error trapping because there are other issues like this that can arise with some of the other combinations of metrics and algorithms.

I always knew that the C-score metric could be improved considerably, but I never imagined the solution would be so compact. Bravo, Dave! Thanks so much for your expertise.


Its never used and since I was already having to recompile everything using this header it seemed reasonable to comment it out (or even remove it).


Also created a new function, validateNameGroup() in 'makeDisplay_internals.R' so that the same checking needed by makeDisplay() and addDisplay() is consistent and has only one set of code (it was duplicated before).


To Do! :grinning:


Should we include the false so if someone looks at the config they can see if they set it to false it took effect? (Just thinking of the dreaded put your -D paramaters in the wrong place effect)


Couldn't this be done cleaner using a regular expression?


Seems strange not to handle the other error case here...!


these are actually two different steps. the initialization of a router is for both client / server.

the start of the client-side router is the beginning of history - also allows a place for a person to hook in and know the client javascript has been initialized properly.


This should read:
```C
enableDebug = true;
shift;
```


I've been thinking this over and I guess if i was going to give anyone honorable mention here i should thank synergydev and codefire.  Yea actually IIRC codefire, codefirex, eos, vanir, linaro.. were all making similar edits to binary.mk before sabermod's hokey snake oil bullshit even existed... not to mention those old commits yielded actual measurable improvements which i'm told is often a complaint for sabermod.  Wierd.. wonder why that is.  Pretty strange since they act like they are the only ones who can read or use makefiles haha


Is this an exception or maybe it should be a Success? Not sure it really matters, but if you have nothing in your flow def this probably isn't the exception you'd expect?


Isn't the function type `recℕ : {X : Type} → X → (ℕ → X → X) → ℕ → X` ? I think you might have missed ℕ


Well, yeah!


The patch works except for the minor fix on line #453.


LGTM +1


i purposely don't have that linked right now so i don't have to start writing the docs for it. i'd rather go back and link it after the first release to github pages, then go back and add the docs for model utils.


Same I think you might have left List ℕ. So the correct type of function is
`recList : {A X : Type} → X → (A → List A → X → X) → List A → X`


Does this need a SQL update for those already testing the server?


develop (lefticus) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2190 tests passed)

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


Yes, go ahead and improve it. I was lazy :)


Circular exits do exist - I've seen them!


Good call. Updated as suggested (new commit coming).


Yep - Coverity hates new String() too.


Also add number-nibble display!


You are correct. Is the mistake in the blog?

On Fri 30 Jan, 2015 7:15 am Ramana Venkata <notifications@github.com> wrote:

> Same I think you might have left List ℕ. So the correct type of function is
>
> recList : {A X : Type} → X → (A → List A → X → X) → List A → X
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/siddhartha-gadgil/LogicTypesSpaces/commit/421e65706da3cb6206e57f415dd1140b0e258e68#commitcomment-9510505>
> .
>


@piccolbo It is not a weakened test in the sense that the intention of the old version is exactly covered by this (10, 6) case (i.e. the case where the last partition is erroneously left off). That said, I didn't realize there's even an R version of QuickCheck the awesome Haskell library! Certainly sounds worthwhile to look into.


technically, it is.  here's the base view's constructor.

```js
 21   constructor: function(options) {
 22     this.options = _.extend( this.options || {}, options || {} );
 23
 24     this.parseOptions(options);
 25     this.name = this.name || this.app.modelUtils.underscorize(this.constructor.id || this.constructor.name);
 26
 27     // parseOptions deals w/ models and collections, but the BaseView will override those changes
 28     Backbone.View.call(this, _.omit(options, ['model', 'collection']));
 29
 30     this.render = this.render.bind(this);
 31   },
```

you can always do things like for the only "required option".
```js
var view = new View()
view.app = this.app;
```


Might have to be a Team Contact, not sure who's "allowed" to update errata after the WG has been disbanded. Maybe @iherman? Of course, if the Wiki allows you to edit it, then I don't see why not. Otherwise, @lanthaler, @msporny  or I can get to it.


Lol


soooo cool!


@khouzam just this tiny modification in e_os.h: https://github.com/iojs/io.js/pull/478/files#diff-1 from memory it was just a link problem for openssl-cli.exe pointing to a missing `__iob_func`, ensuring that block wasn't invoked for 2015 does the trick


HAHAHA this is what happens when you get interrupted brain dumping docs. :p thanks for the clean up :dancer:


Nice! Apache Commons does everything! Updated and all the unit tests still pass so I'm going to push this as well. It's nice to get rid of all the crazy string parsing.


Added my solution. Thanks.


Thanks you answer, here error message:

 Information about the domain could not be retrieved (1355).
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.DirectoryServices.AccountManagement.PrincipalOperationException: Information about the domain could not be retrieved (1355).
Source Error:
Line 186:                using(ADUser user = ADUser.FindByIdentity(context, userId))
Line 187:                {
Line 188:                    foreach(var grp in user.GetAuthorizationGroups())
Line 189:                    {
Line 190:                        // We don't want to show the Users and Domain Users groups

Source File: c:\Users\thanhthuy\Downloads\ADWeb-master\ADWeb-master\ADWeb.Core\ActiveDirectory\ADDomain.cs    Line: 188


should return err?


when i click view user , it has that error!


下面这三种情况什么时候会是err?


rax retry tests


And this is the optimisation that speeds up deleting a group or whole area of interconnected rooms :smile:


@krukow Changes in this file should never be committed.  They are updated at compile time.


The 69K room test data map I was using has a room with an id of 0!


I am nobody to judge intentions, but running the same test on more inputs catches more bugs, that's what I meant by "weakened". It's just logic but there is also a paper about it (something like "kolmogorov complexity justifies software engineering heuristics".) We (Revolution) just released the quickcheck package, let me know if you have any questions.


the curly brace should be on the next line


Yes. I wanted to make sure that before i submit PR.


Just curious. Was this the source of the pain?


This should not have been sync'ed


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/805


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1990](http://teamcity.monogame.net/viewLog.html?buildId=5127&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 368, ignored: 6 Build time: 00:16:39


1: M111 unrecognized component name: oauthclient


Doesnt make much difference, but I might have done something like

```python
if latest_data is None:
    continue
````

Which means the block as a whole is less indented, which is slightly more readable to me. Not a big issue though.


Please make sure to credit liquidsmooth as your base in your rom threads


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1991](http://teamcity.monogame.net/viewLog.html?buildId=5129&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


@saghul On android 5.0 the function `pthread_cond_timedwait_monotonic_np` was removed which breaks mapbox - gl - native with arm64 devices like the nexus 9


Oh, I'm blind. De-duped in 83e7a31 ;)


Latest compiled 4.9.3 GCC has files bigger than 100MB, so it can't be pushed to github :(.
cc1, cc1plus and lto1 are bigger :(.


I like that, so we have pure immutable object


Don't know ... But it seems `@access` isn't really useful.


I Like This Homework


just to clarify, so an adapter can now call build_absolute_uri(site, location) and get back the uri as `http://site.domain/location` instead of `http://request.host/location`, right?


I think 3.0 will.


@borshop merge


```ruby
set :output, "log/cron_log.log"
```


Thanks for the submission.

For consistency, can you please also add tests for the new c interface methods to ctest.c:CompileTest?

Thanks

Andy


Worth documenting in all of the places where preparedAttachments is passed around that it's a map of <id,rev> → list of attachments. I know I originally came up with that structure but now that it's being passed around it's not at all intuitive.


just remove unused code if not needed :)


Indent for this block wrong


delete these un-needed newlines


@gavanderhoorn Now it loads the right macro and crashes at launch


:+1: Happy to see your first GitHub contribution! :tada:


Hmm. Will need reverting over on the php-build repo for 5.2.17 too then.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1991](http://teamcity.monogame.net/viewLog.html?buildId=5129&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests passed: 303, ignored: 6; exit code 1 (new) Build time: 00:12:47


Done. It'll be in the next published version.


Hi, thank your for taking part.

Well, this variable needs to be a date in the past because the current default update-interval value is once a day.
So, if somebody is running osu!Sync for the first time and the variable isn't in the past the application won't check for updates which I think is quite important to do on the first startup.


This is most probably wrong. We don't want absolute paths anywhere.


No need for parenthesis. And the documentation that should be pointed here is the doc/static, which should be enhaced to explain what sys/static.sh does for example


Hint, relative links are supported:
```
[Code Club UK fomat](/CodeClub/curriculum_documentation/blob/master/projects.md#2-write-your-project)
```
Saving wooping 12 characters! :see_no_evil:


I don't know reason but LibreOffice developers are migrating from macros foreach directly to c++11-for loop.

Thanks for information! Yep, since I knew it, I am smarter little bit ^_^.


Maybe near https://github.com/OP-TEE/optee_os/blob/master/documentation/build_system.md#cross_compile-cross-compiler-selection?
Still, I think a short comment in sub.mk would help and even be sufficient...


Please add whoops to common lv5 install. I loved that look.


1: M111 unrecognized component name: oauthclient


Tried to move that logic outside a bit and add caching of values as per https://github.com/quantopian/zipline/commit/d3667e0e40f2356b8fd1261ae349f4334eb5cbbc#commitcomment-9507822. Inheriting from Series might cause problems but it was the easiest way to get the numpy array functionality like multiplication.


Yes, it surprised me too. The problem is that the attribute may not be a String. For example `<f:with bean="${bean}">`. In this case, bean?.toString() may be an empty string. The fix isn't ideal, but it restores the behaviour previous to this change:
https://github.com/grails-fields-plugin/grails-fields/commit/fc1d4fc70dd23bd04e885dab58786a88462e75fd#diff-365ac56876ad899acb12b96b97eb3efaL226


LGTM

Thanks for this!


Thanks @dalejung. Took a crack here: https://github.com/quantopian/zipline/commit/101b563d5fe14ebc83aaa3369deac8ae97c6a509


OK, I'll add a short note in sub.mk


Isn't that 1GHz ARM?


Use of API, and apparently, this shouldn't be too hard to reproduce, I'm busy on a "steps to reproduce" that should be something similar to:

1. create a content structure A / B / C / D
2. move "C" directly under "A"
3. remove "B" without using the trash
4. access "C" while having a breadcrumb on it (I suppose that simply using twig's `path` on "C" would trigger the issue)

It appears that there is still somehow/somewhere a *reference* to "B".


Not a big deal but FYI you can specify the event and the keycode directly in the match arm using:

```rust
Event::KeyDown { keycode: KeyCode::Escape, .. } => break,
```


I integrated this fix but I'm still having an issue
My event handlers are not called
And while I'm in Readonly mode, on dble-click the cell still pass in edit mode


No, because this checks for any string that contains only blanks. `length_filter` could be used with min > 1, but it doesn't accomplish *exactly* the same goal.

Also, it turns out that this is expected behavior from ICU (see figure 1: http://www.unicode.org/reports/tr29/#Word_Boundaries)


The wakeonlan command isn't ran with sudo. `passthru ('wakeonlan '.$mac.'');`
But it does allow me to inject commands to be ran as www-data, which with the entry in sudoers is able to run any and all commands as any user via the sudo command.

This is not the first time I see such security issues, and large corporations make the same errors. For example, I can inject commands that are ran as root via the web interface of a consumer router. (said router is running linux under the hood, and said hole is only availavle when loged in as the admin user of the web interface).

The problem is lack of sanitization of user input. Idealy you would never execute any command containing a string that could come from an external location, like user input in a web form.

You could sanitize the input of a MAC address by matching it against a regex checking if it's a valid mac. ie. containing only hexadecimal numbers and colons.

Still, allowing the web server to run any command as root defeats the purpose of running it as a separate user in the first place, and you could just as well run apache and php as root. Security by obscurity is still not security.


Stimmt das? War in der alten Version der Zusammenfassung, steht aber nirgends so im Skript. (Habe ich zumindest nicht gefunden).


foobar


ID based call is not supported?


Thanks for spotting the typo! That's just the instruction text so no harm on it. The actual value is here: https://github.com/KanoComputing/kano-settings/commit/4b202e6fb4258d64404f9caf2b0a072ed9416e03#diff-4cf9d584644fe467b248b4b2cd6056bfR51


first comit


Would be `REFERENCES events(event_id` no?


Il reste une action à faire dans l'entité Seminar à propos de "state" (à relier avec State.. je laisse Valérie le faire pour s’entraîner !)


I would like a method in here that returns a boolean. You should be able to use this method to see if the server version support titles. You could call it something like `isTitleAvailable`.


This class edit is for style purposes ;)


asdf


fdsaaaaa


That sounds more plausible!


86121908_copy_schedule (macumber) - Win64-Windows-7-VisualStudio-12: OK (2174 of 2190 tests passed)

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


fdsa


Empty line to be removed.


zaebok!)


You could just catch Exception.


Print ASR job


>  Should clients be able to satisfy routing requests?

Dont think so. let's keep it simple.


ok


Setting fields accessible is a good idea. Not only for accessing them to be possible but I heard that it offers a performance boost as well. So I'm suggesting you set all fields accessible for the sake of performance.


Wrong indent.


You could just catch the Exception class.


Same thing here. Just catching the Excpetion class would be cleaner.


ResizedAsset collection is designed for only ImageAssets. There is no need to include in DocumentAsset & VideoAsset.


(minor change: I used `config.context`, that one also includes possible globalConfig value of context, options only includes the values from the current $.soap call)


return ASRSite object as its user facing object.


665px надо


This breaks booting with an mdadm array as root file system, because within the initrd, `/sbin/mdadm` is called instead of the full store path (at least I think, haven't checked, can't check right now, late for train).


will not work for IE and FF. Here is a cross browser solution:

```
            var blob = new Blob([data], { type: 'text/csv;charset=utf-8;' });
            if (navigator.msSaveBlob) { // IE 10+
              navigator.msSaveBlob(blob, options.filename);
            } else {
              var link = document.createElement("a");
              if (link.download !== undefined) { // feature detection
                // Browsers that support HTML5 download attribute
                var url = URL.createObjectURL(blob);
                link.setAttribute("href", url);
                link.setAttribute("download", options.filename);
                link.style = "visibility:hidden";
                document.body.appendChild(link);
                link.click();
                document.body.removeChild(link);
              }
            }

```


Indeed. Actually it's the IDE collapsing docs :)


There is no need to seperate "Asset" to DocumentAsset and VideoAsset.


Like I said before, I really don't see why you insist on running the same code for navigation events and click events.   They are completely different things.


What events are you having issues with? onclick events created in studio?
And what version of wakanda are you using?

Make sure you have updated widget.js also,incase you are using old version
https://github.com/vegarringdal/GridView/blob/master/widget.js#L206

About the readonly, the grid allows you to edit cell, but it will not save

Im been planning to make this better, just havent gotten any further yet

If you do not was user to be able to edit text at all while readonly you could remove you could remove commented "return false"
https://github.com/vegarringdal/GridView/blob/master/vr-widget.js#L656

Plan is to actually add a "readonly" attribute here, if readonly..
https://github.com/vegarringdal/GridView/blob/master/vr-widget.js#L618


Use of ternary operators like this can be a little bit hard to read.
For readability I would do this instead:
```java
if (mainTitle != null) mainTitle = Txt.parse(mainTitle);
```


Why is this in the fix-321 branch? It seems to be unrelated to fixing #321.


FabricType is used in server as well.


this is just refactored


I changed this line to:
```
config.success.call(this, response);
```
Now within the success function `this` will be set to whatever you put into `context`, so your callback becomes something like:
```
var callback_success = function(xml_response) {
            var ticket = this.self.otrs_queues[context.queue_name].tickets[context.ticket_id];
```


Just catch Exception?


Thanks a lot! But didn't you plan to use Jekyll Scholar? Also, the Marburg bibliography wasn't used directly, but divided into sections with a set of bib2bib-based scripts (which I did backup from Marburg).

A few bug reports:

- The pipeline misunderstands some TeX-isms, like \lambda or \.
- Plus, Markus Kreutzer didn't publish at Haskell symposium, but see below:

* Layout-sensitive Language Extensibility with SugarHaskell

by Sebastian Erdweg, Felix Rieger, Tillmann Rendel, and Klaus Ostermann

In Proceedings of Haskell Symposium 77(3): . ACM Press, 2012.

* Statische Analyse von Produktlinien

by Markus Kreutzer

In Proceedings of Haskell Symposium 77(3): . ACM Press, 2012.

The whole thing likes to repeat all fields if they're not overriden — "77(3):" repeats over quite a few more publications. I'm guessing that's because `capture` is not `let` but something more dynamically scoped/imperative/whatever.


Since you are not doing anything with `context` within `SOAPObject` I removed this one...


* Added a simple Filter input, that will show only the desired property name
![screen shot 2015-01-30 at 12 31 55](https://cloud.githubusercontent.com/assets/946884/5974365/4b17156e-a87c-11e4-8ed1-cf6084c04560.png)


hi eiriklv,
http://facebook.github.io/react/docs/component-specs#mounting-componentdidmount
```
Invoked once, only on the client (not on the server) ...
```
can we spare this check here or is there any other concern?


......and this one.


86121908_copy_schedule (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2160 of 2190 tests passed)

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


fast-forwarding develop to borshop-integration-1065-bugfix/depgraph-ee = 5d772ef1


Reviewed ! looks fine.


Because a large part of the code is the same in both cases, and if I wouldn't use a shared method for both you'd complain about code duplication :-) Anyway, I wouldn't think this is a critical issue.


cls-4 is not used anymore but still part of the style sheet in `<style>`.


This is the piece of code I refered to as 3) in my message. The question is, what does it do? Or better, what does it solve? Was there a problem that you had that needed this?


Relative links mean if someone is reading the README in a local clone, the link won’t work. Personally, I’d always use an absolute link (but I followed the existing convention here, and removed the protocol).


If it is in the map it might be not a leaf, just somewhere in the chain; and if so we just remove this one and keep all children; otherwise if it is actually the last one we can remove it.
In the case we delete something not there, we do not delete anything (this case should not happen here anyway)


1: M100 needs more reviewers
1: M111 unrecognized component name: oauthclient


24:1: I100 Imports statements are in the wrong order. from flask should be before import six
27:1: I100 Imports statements are in the wrong order. from functools should be before from flask.ext.login
27:1: I201 Missing newline before sections or imports.
28:1: I201 Missing newline before sections or imports.
30:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
43: D103 Docstring missing
49: D103 Docstring missing


23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.


23:1: I101 Imported names are in the wrong order. Should be session, url_for
23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
28:1: I201 Missing newline before sections or imports.
29:1: I201 Missing newline before sections or imports.
30:1: I201 Missing newline before sections or imports.


26:1: I101 Imported names are in the wrong order. Should be BadData, TimedJSONWebSignatureSerializer
26:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.local
33:1: I101 Imported names are in the wrong order. Should be disconnect_handlers, handlers, oauth, signup_handlers
34:1: I101 Imported names are in the wrong order. Should be authorized_default_handler, disconnect_handler, make_handler, make_token_getter, oauth_logout_handler, set_session_next_url


1: D400 First line should end with '.', not 's'
1: D205 Blank line missing between one-line summary and description
28:1: I100 Imports statements are in the wrong order. import base64 should be before import time
32:1: I100 Imports statements are in the wrong order. import os should be before import sys
33:1: I100 Imports statements are in the wrong order. import inspect should be before import os
36:1: I101 Imported names are in the wrong order. Should be FancyURLopener, quote, quote_plus, urlencode
37:1: I201 Missing newline before sections or imports.
38:1: I101 Imported names are in the wrong order. Should be escape, parse_qs, parse_qsl
38:1: I100 Imports statements are in the wrong order. from cgi should be before from six.moves.urllib.parse
38:1: I201 Missing newline before sections or imports.
39:1: I201 Missing newline before sections or imports.
48:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.local
49:1: I101 Imported names are in the wrong order. Should be HASHLIB_IMPORTED, md5, sha1
51:1: I100 Imports statements are in the wrong order. from invenio.utils should be before from invenio.utils.text
54: D400 First line should end with '.', not ','
54: D205 Blank line missing between one-line summary and description
76:13: B901 blind except: statement
82:28: E261 at least two spaces before inline comment
86:13: B901 blind except: statement
93:13: B901 blind except: statement
97:30: E261 at least two spaces before inline comment
102:29: E261 at least two spaces before inline comment
110:1: I101 Imported names are in the wrong order. Should be urljoin, urlparse
110:1: I100 Imports statements are in the wrong order. from urlparse should be before from invenio.utils
110:1: F999 redefinition of unused 'urlparse' from line 37
111:1: I201 Missing newline before sections or imports.
111:1: F401 'url_for' imported but unused
113:1: E302 expected 2 blank lines, found 1
118:12: E127 continuation line over-indented for visual indent
131: D205 Blank line missing between one-line summary and description
204: D400 First line should end with '.', not 'L'
220: D290 White spaces around docstring should be trimmed.
220: D205 Blank line missing between one-line summary and description
240: D103 Docstring missing
265: D290 White spaces around docstring should be trimmed.
265: D202 No blank lines allowed *after* function docstring, found 1
265: D400 First line should end with '.', not 'n'
265: D205 Blank line missing between one-line summary and description
288: D401 First line should be imperative: 'Create', not 'Creates'
288: D205 Blank line missing between one-line summary and description
308:65: E502 the backslash is redundant between brackets
310:33: E127 continuation line over-indented for visual indent
313:33: E127 continuation line over-indented for visual indent
320: D401 First line should be imperative: 'Create', not 'Creates'
321:9: E128 continuation line under-indented for visual indent
322:9: E128 continuation line under-indented for visual indent
435:13: E128 continuation line under-indented for visual indent
436:13: E128 continuation line under-indented for visual indent
438:13: E128 continuation line under-indented for visual indent
439:13: E128 continuation line under-indented for visual indent
446: D205 Blank line missing between one-line summary and description
456:1: E302 expected 2 blank lines, found 1
456: D400 First line should end with '.', not 'e'
456: D205 Blank line missing between one-line summary and description
485:1: E302 expected 2 blank lines, found 1
485: D401 First line should be imperative: 'Create', not 'Creates'
485: D400 First line should end with '.', not ':'
485: D205 Blank line missing between one-line summary and description
499:70: E502 the backslash is redundant between brackets
501:33: E127 continuation line over-indented for visual indent
504:29: E127 continuation line over-indented for visual indent
511: D290 White spaces around docstring should be trimmed.
511: D202 No blank lines allowed *after* function docstring, found 1
511: D400 First line should end with '.', not 's'
523: D400 First line should end with '.', not ')'
523: D205 Blank line missing between one-line summary and description
523: D208 Docstring is over-indented
539:42: E261 at least two spaces before inline comment
543:14: E261 at least two spaces before inline comment
553:1: E302 expected 2 blank lines, found 1
553: D400 First line should end with '.', not 'm'
553: D205 Blank line missing between one-line summary and description
565:9: B901 blind except: statement
571: D400 First line should end with '.', not 'o'
571: D205 Blank line missing between one-line summary and description
582: D203 Expected 1 blank line *before* class docstring, found 0
586: D102 Docstring missing
589: D400 First line should end with '.', not 't'
597:1: E302 expected 2 blank lines, found 1
597: D400 First line should end with '.', not 'e'
597: D205 Blank line missing between one-line summary and description
606:1: E302 expected 2 blank lines, found 1
606: D202 No blank lines allowed *after* function docstring, found 1
606: D400 First line should end with '.', not 'g'
606: D205 Blank line missing between one-line summary and description
636: D401 First line should be imperative: 'Return', not 'Returns'
636: D202 No blank lines allowed *after* function docstring, found 1
636: D400 First line should end with '.', not 'e'
636: D205 Blank line missing between one-line summary and description
674:57: E502 the backslash is redundant between brackets
675:55: E502 the backslash is redundant between brackets
683:29: E127 continuation line over-indented for visual indent
684:29: E127 continuation line over-indented for visual indent
685:29: E127 continuation line over-indented for visual indent
691: D401 First line should be imperative: 'Compute', not 'Computes'
691: D400 First line should end with '.', not '4'
691: D205 Blank line missing between one-line summary and description
708:13: B901 blind except: statement
721:6: E111 indentation is not a multiple of four
733: D202 No blank lines allowed *after* function docstring, found 1
800:9: B901 blind except: statement
809:1: E302 expected 2 blank lines, found 1
809: D203 Expected 1 blank line *before* class docstring, found 0
809: D400 First line should end with '.', not '"'
809: D205 Blank line missing between one-line summary and description
809: D300 Expected """-quotes, got '''-quotes
827: D400 First line should end with '.', not ')'
827: D300 Expected """-quotes, got '-quotes
838:33: W602 deprecated form of raising exception
852:13: E129 visually indented line with same indent as next logical line
861:5: E303 too many blank lines (2)
861: D203 Expected 1 blank line *before* class docstring, found 0
861: D204 Expected 1 blank line *after* class docstring, found 0
861: D400 First line should end with '.', not '6'
861: D300 Expected """-quotes, got '-quotes
866: D203 Expected 1 blank line *before* class docstring, found 0
866: D204 Expected 1 blank line *after* class docstring, found 0
866: D400 First line should end with '.', not '1'
866: D300 Expected """-quotes, got '-quotes
870:1: E302 expected 2 blank lines, found 1
870: D401 First line should be imperative: 'Append', not 'Appends'
870: D290 White spaces around docstring should be trimmed.
870: D400 First line should end with '.', not 'e'
870: D205 Blank line missing between one-line summary and description
870: D208 Docstring is over-indented
884:1: E302 expected 2 blank lines, found 1
884: D401 First line should be imperative: 'Return', not 'Returns'
884: D400 First line should end with '.', not ':'


This one is really nice and clean


1: M111 unrecognized component name: oauthclient


24:1: I100 Imports statements are in the wrong order. from flask should be before import six
27:1: I100 Imports statements are in the wrong order. from functools should be before from flask.ext.login
27:1: I201 Missing newline before sections or imports.
28:1: I201 Missing newline before sections or imports.
30:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils


23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
38:9: F821 undefined name 'user'


also changed with call:
```
soapObject = options.data.call(options.context, SOAPObject);
```


Hors de question que ça ne soit pas dans le projet ouaich


You really need that? print?


> I think we want to add the CSS to style.css so that <progress> looks the same in all browsers

I have really bad experiences with that. Styling the progress bar equally sucked even in only Chrome and FF, I tried that in another project of mine. I would strongly recommend to let the browser handle this and live (but stay sane ;) ) with the inconsistency.

I'm ok with everything else.


Why we provide a general approach to validate skip & take params? There is a totally different validation logic in AutoComplete.GetAsync() method with same parameters?


http://www.kartudewa.com/
http://www.idolapoker.com/
http://www.prediksihoki.com/


I'm not sure you understood me. You are missing a back tick in the README.md file which makes the whole sentence to look like a code.


need to think more before coding!!!


I think it would be better to update translation string to use `%d` instead of `%s` so that no conversion is necessary.


Sorry, i'm going to nitpick again. These can be `unless: :covers_whole_of_uk_true_or_nil?`

Also, on the naming of the predicate, it makes sense in the technical domain, but not in the business domain. How about switching to `if` and naming it `requires_geographic_coverage?`


Well, the problem is that the way the CSS is referenced now, only IE will get the CSS for the polyfilled `<progress>`, so we need to at least add the CSS to our `style.css`. As far as I can tell, that will not affect browsers that support it natively because they don't have the extra divs the polyfill adds (and that are styled by the extra CSS).

Then again, we might want to think about adding a way to load polyfills and their required assets based on feature detection anyway. Unfortunately, the only script loader I have experience with for that is YepNope, and that [is deprecated](https://github.com/SlexAxton/yepnope.js#deprecation-notice) …


userdata -> feedback


I made all the update, still doesn't work

using Wakanda 10 beta from 28 january


Will you make a pull request?


Patch is fine besides the minor question.


Back then a lot was missing from it, so it was easy and fast to make new
releases.

On Friday, January 30, 2015, NicklausW <notifications@github.com> wrote:

> To think, this same program has had as little as two days between releases
> before.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vhelin/wla-dx/commit/1e1c09c3246e6c36b1ff98591dd8146a77fed0ed#commitcomment-9510825>
> .
>


Added these (long) custom properties which have been missing from revLibUrl for a while. When we've made the changes to standalone building to allow script-only stacks to have resource files, these can be moved into text files instead.


True would be nice.


True


It does? Sure.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 941](http://build.osmsharp.com:8080/viewLog.html?buildId=4203&buildTypeId=bt16) is now running


If you think so


Look at what line I posted my comment on:

~~~~
-#if !defined(__ANDROID__)
+#if !(defined(__ANDROID__) && defined(HAVE_PTHREAD_COND_TIMEDWAIT_MONOTONIC))
~~~~

That chunk of code won't be compiled on Android, so why the added check?


Woops


It looks pretty good. Is it possible to do something like this with a Rule:

```java
public class MyTest {

   @Inject
   MyEJBService myService;

  @Test
   public void shouldInjectService() {
      myService.callOtherServiceAndPersistAnEntity();
   }

```

This is why we wanted to use the Junit-Runner. But, if it is possible to do that with a rule (the injection of the service), then we can use it :-)


My bad


haha! Vou fazer uns commits hoje :)


86121908_copy_schedule (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 163](http://192.168.30.225:8111/viewLog.html?buildId=692&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 163](http://192.168.30.225:8111/viewLog.html?buildId=692&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


I prefer a very spaced code, it's more readable. I don't think there's any flexibility with PSR :/


It looks sane but is quite complex.


Can you look if php-cs-fixer align `$throws` tag ?
According to [phpdoc.org](http://phpdoc.org/docs/latest/references/phpdoc/tags/throws.html) the tag name is throw**s**.


s/ii/end/ :)


sure, if it helps.


Or you change all other ends back to iis :)


Looks good to me in general. I still would like to see the style sheet cleaned up after collapsing. Can be done in a different patch though.


@saghul well before with the old versions of Android `pthread_condattr_setclock` did not exist, hence why they skip it with `#if !defined(__ANDROID__)` and why you need to use `pthread_cond_timedwait_monotonic_np`

In Android 5.0 you need to use `pthread_condattr_setclock` with `pthread_cond_timedwait`

The `HAVE_PTHREAD_COND_TIMEDWAIT_MONOTONIC` is only defined prior to Android 5.0, they removed it to provide a way to easily detect if you should use the old or the new method.

Thus this line says if Android and *old Android* do not run this function. If *new Android* do run this function.


- The commit message mentions a bug fix for `cache_maintenance_l1()` but this does not seem to be modified by the patch?
- I would suggest to split the common part and the STM part, in two commits:
  - "pl310: Refactor arm_cl2_cleanbypa(), arm_cl2_invbypa(), arm_cl2_cleaninvbypa()"
  - "plat-stm: cache_maintenance_l2(): use area funtions"


Yes, the css should not bother other browsers. Just the 10em width I added.

And yes, a proper way to load them only when needed starts to become really nice.


Just tested, the onRowClick event worked when I tested it on a very simple solution just now.
What events have you added?

![image](https://cloud.githubusercontent.com/assets/2901416/5974608/70dd3fea-a876-11e4-847c-2e602bcbd256.png)


OK I found out another way. Just:
```python
def mock(backup, module, attribute, log=None, ret=None):
    if module.hasattr(attribute):
        backup.append((module, attribute, getattr(module, attribute)))
    else:
        backup.append((module, attribute))
    setattr(module, attribute, mock_function(...))
```


So, 006bb855f45434e5cf2c2a8a2bdec1267d21d6b0, 11bbf39cc6316f6f3649f601108aff8610d49c80 and 147a9ab43e64f42a0a054ee259744999c15a3a9d are what I can do here. I also tweaked the CSS a little to save some bytes and to make sure it *really* does not affect browsers that support `<progress>` already.

All this is tested in IE 9 and Chrome, works well in both. If you guys don't have any other concerns about this, I'd say this is good to merge.


Better to use target user instead of enabling DEXPREOPT on target userdebug.

Not sure if we want this.


This could help you:
http://static.cegui.org.uk/static/TaharezLookProperties.html#VerticalScrollbar

I haven't look at the vertical scrollbar properties yet, though.


Note that this commit is sort of broken. There is no consistency in the typing of the port variable. It is declared a char*, but is assigned as if it were an int in a few places, and used in the sprintf() call as if it were a short int. Does this work at all?


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.941](http://build.osmsharp.com:8080/viewLog.html?buildId=4203&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 505 Build time: 0:0:0


Great, thanks! I still want to add error display, probably sunday, but that should not change anything we worked on till now.


I think it would be nice... if you don't, I'll merge on my fork and submit it :)


Ok, let's do that way for the time being...


I took the liberty of closing #194 already since it will be covered by #279 now.


TeamCity Ylab :: zmactep/ylab [Build 188](http://192.168.30.225:8111/viewLog.html?buildId=693&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 136 Build time: 00:12:18


\the [W]


"a full cartridge"


I think, we will have to write validators ourselves in this case. We can't have fancy usernames, because it could break integration with Mongoose.


delete


Ha! Of course, I've been seeing `unless: {}` for days now, in my sleep, too. :D


Any hacked init for kitkat!


delete file


OK.


fixed


Thanks for the link! :)

From your link btw, sinbad is saying:
`In fact smoothing is off by default. Any erraticness is likely to be because of timer precision or single-frame hiccups. The smoothing is there to help eliminate those sorts of things and you should set the frame smoothing time to a number that works well for you.`

I'll have to check but I guess the smoothing is still off by default, since the smoothing itself must be a little bit more time consuming. That would make sense to me at least.

In any case, if you want, we can make tries and adapt accordingly but I'd rather not do that like it is done, since, btw, even the comment hasn't been adapted. And this shows it was a quick-fix. Please, revert for now and I'll gladly test all that in someone's next PR about this.

Deal?


It seems there is forgotten commented code here.


I don't think there's any code duplication.   What part do you think would be duplicated?


Looks good.


@nirvdrum @eregon Array#each returning an enumerator - turned out quite simple


If I'm reading this correctly the `itemDel` is set to "." on line 4666, but `cTopLevelDomains` is comma delimited.


Not anymore :) https://github.com/Kdyby/Monolog/blob/master/docs/en/index.md


@JeroenDeDauw Constraint is currently set to `mediawiki/semantic-media-wiki": "~1.9,>=1.9"`


is there a way to add handle_errors as errback here? seems simpler without passing via handle_crawl_errors


Yeah, the first call may raise (okay, actually not but the compiler doesn't track that yet I guess), so that actually makes sense.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1992](http://teamcity.monogame.net/viewLog.html?buildId=5131&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Now the gadget has become common among the people of allkinds. People of almost all fields can use their iPhone for their studies, business, entertainment, exploration, etc. with a number of available apps in the app store which are helpful for their purpose.

This made the device more common. This eventually increased the scope of <a id="qlink_po0bzjzyct" href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/" data-link-text="iPhone app development Los Angeles" data-cke-saved-href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/">iPhone app development Los Angeles. iPhone is one of the fastest growing application development platform compared to the other similar platforms. Even though the device it a bit costlier than the other devices of similar specification Apple allows the users to make maximum use of it by allowing them with a great access to a number of downloadable apps in their website.

Now there are a number of <a id="qlink_zxs40p7cv6" href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/" data-link-text="iPhone app developer Los Angeles" data-cke-saved-href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/">iPhone app developer Los Angeles companies for iPhone applications. Many of the PC game makers have changed their major priority from the PC platform to the Apple gaming platforms after realizing the fact that it is the more popular and easier way for making revenue. A number of apps are coming to the platform every day. This makes a tough competition between the developers. The companies are being developing more innovative and exciting apps for including more fun to the device rather than concentrating on the business purposes.

This clearly shows two present conditions. First: favorable to the companies seeking a place in the business <a id="qlink_dqi608vwze" href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/" data-link-text="Iphone app development" data-cke-saved-href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/">Iphone app development area. There is more scope in the business Iphone app development section than the game area. But the other is a risk factor for the companies in the other sections. More companies are reaching the app store with their new products every hour. This makes a tight competition in the fun section.

If you are reaching the app store with a new app, it must contain a lot of fun and it should be arranged well to consider people of almost all age. Now the apps are downloaded by all type of people equally. The latest apps in the app store are designed for all type of people like children, youth, business people, and old ones.

The online app store of Apple is now over crowded with over seven hundred thousand applications. This gives the needy a thousand options. It also increases the confusion in the downloader for selecting the app for the download. So what the needy really do is reading the comments and reviews as well as considering the ratings of the app in the app store.

For getting a good feedback from the users it should be fascinating by several means. The usage levels of power and RAM are the main points generally people consider. The availability of options, connections with various websites while using the app, customer care supports, and problem reporting facilities are the major options people considers before click on the ‘download’ option.

Experts are working in ClapCreative which is <a id="qlink_a0kzn45vpb" href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/" data-link-text="iPhone app Development Company Los Angeles" data-cke-saved-href="http://clapcreative.com/services/los-angeles-mobile-app-development/iphone-ipad/">iPhone app Development Company Los Angeles offers to create Iphone application with cost effective solutions for more details send your inquiry to info@clapcreative.com.


this can happen only if user overriding this method allows passing of wrong parameters to method, right?


To count by type of eslint errors:

```bash
grunt | grep error | grep -oe ' [a-z-]*$' | sort | uniq -c | sort
```


PS: I see in another comment you wrote:

> If I put it directly in the click handler instead of this function shared by both click and keynav interaction I'd need to repeat (duplicate) the check of selection mode, renderer target etc.

But as far as I can tell all you need to duplicate is the line `if (this.selectionMode === "single" )`,  i.e.:

```js
this.list.on("click", function () {
   if (this.selectionMode === "single" ) {
       this.defer(function () {
		// deferred such that the user can see the selection feedback
		// before the dropdown closes.
		this.closeDropDown(true/*refocus*/);
	}.bind(this), 100); // worth exposing a property for the delay?
   }
}.bind(this));
```


Новая строка или вставка коммента? x)


Can you please provide me the original code that you have downloaded from
http://h264.code-shop.com/download/mp4split-2.2.0.tar.gz  ?  My email ID
is prashanta.tezu@gmail.com


Sorry for inactivity, IRL problems...

Thank you for your interest in the project.

I agree, the compiled libs are too big and need to be downloaded in another way. I created a directory on dropbox (maybe in the future there's gonna be a proper web site).

The file structure change, dunno what to say. There's very few files atm, and adding more directories for just 7 cpp files seems a little too much, but that's an opinion.

On the matter of IDE project files, I made a lot of tries with different IDEs, (visual studio, codeblocks, codelite, netbeans etc) and each have different project file systems and everything gets really messi if you try to arrange them all in the OSI project. This seems easy but it's not: there are 4-5 big IDEs and all (excluding vstudio) must be made to work on all 3 operating systems... the horror :) So at the moment, i just excluded anything i did with the IDEs, and everyone does whatever he wants with the IDEs. We can try (again in my case) to make everything work, tho, and expand the git to have IDE project files.

I am a noob at the makefiles, so if you want to do something about them, you're free to do so.

Recently changed the openGL directory to oGL because there might be problems under MacOS: the include file is the same <OpenGL/BlaBla.h> and renaming it to oGL, excludes this possible problem


There no need for that , double quotes inside the lang strings are allowed.
here is the example of that same string
http://screencast.com/t/2JwL0aigJlH


restrict to office/VPN


minor: spaces needed around RIPF


LGTM