1: D400 First line should end with a period (not 'd')
1: D205 1 blank line required between summary line and description (found 0)
35:1: E402 module level import not at top of file
36:1: E402 module level import not at top of file
37:1: F401 'urlparse' imported but unused
37:1: E402 module level import not at top of file
38:1: I100 Imports statements are in the wrong order. import socket should be before import urlparse
38:1: F401 'socket' imported but unused
38:1: E402 module level import not at top of file
39:1: I100 Imports statements are in the wrong order. import smtplib should be before import socket
39:1: F401 'smtplib' imported but unused
39:1: E402 module level import not at top of file
40:1: I100 Imports statements are in the wrong order. import re should be before import smtplib
40:1: E402 module level import not at top of file
41:1: I100 Imports statements are in the wrong order. import random should be before import re
41:1: F401 'random' imported but unused
41:1: E402 module level import not at top of file
42:1: I100 Imports statements are in the wrong order. import datetime should be before import random
42:1: E402 module level import not at top of file
44:1: F401 'Request' imported but unused
44:1: E402 module level import not at top of file
45:1: I201 Missing newline before sections or imports.
45:1: F401 'iteritems' imported but unused
45:1: E402 module level import not at top of file
46:1: I100 Imports statements are in the wrong order. from socket should be before from six
46:1: I201 Missing newline before sections or imports.
46:1: F401 'gaierror' imported but unused
46:1: E402 module level import not at top of file
47:1: I100 Imports statements are in the wrong order. import os should be before from socket
47:1: E402 module level import not at top of file
48:1: I100 Imports statements are in the wrong order. import binascii should be before import os
48:1: E402 module level import not at top of file
49:1: E402 module level import not at top of file
51:1: E402 module level import not at top of file
52:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_ACCESS_CONTROL_LEVEL_GUESTS, CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_ACCESS_CONTROL_LIMIT_REGISTRATION_TO_DOMAIN, CFG_ACCESS_CONTROL_NOTIFY_ADMIN_ABOUT_NEW_ACCOUNTS, CFG_ACCESS_CONTROL_NOTIFY_USER_ABOUT_NEW_ACCOUNT, CFG_BIBAUTHORID_ENABLED, CFG_CERN_SITE, CFG_INSPIRE_SITE, CFG_SITE_ADMIN_EMAIL, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_WEBSESSION_ADDRESS_ACTIVATION_EXPIRE_IN_DAYS
52:1: F401 'CFG_WEBSESSION_ADDRESS_ACTIVATION_EXPIRE_IN_DAYS' imported but unused
52:1: F401 'CFG_SITE_NAME_INTL' imported but unused
52:1: F401 'CFG_SITE_RECORD' imported but unused
52:1: F401 'CFG_BIBAUTHORID_ENABLED' imported but unused
52:1: F401 'CFG_INSPIRE_SITE' imported but unused
52:1: F401 'CFG_SITE_SECURE_URL' imported but unused
52:1: E402 module level import not at top of file
53:6: E126 continuation line over-indented for hanging indent
76:1: E402 module level import not at top of file
77:1: E402 module level import not at top of file
80:1: I101 Imported names are in the wrong order. Should be gettext_set_language, wash_language, wash_languages
80:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.serializers
80:1: F401 'wash_language' imported but unused
80:1: F401 'wash_languages' imported but unused
80:1: E402 module level import not at top of file
81:1: E402 module level import not at top of file
82:1: E402 module level import not at top of file
83:1: E402 module level import not at top of file
84:1: I201 Missing newline before sections or imports.
84:1: E402 module level import not at top of file
86:1: E402 module level import not at top of file
97:1: I100 Imports statements are in the wrong order. from invenio_access.local_config should be before from sqlalchemy.exc
97:1: E402 module level import not at top of file
110:1: E302 expected 2 blank lines, found 1
110: D400 First line should end with a period (not 's')
110: D208 Docstring is over-indented
126:1: E302 expected 2 blank lines, found 1
126: D202 No blank lines allowed after function docstring (found 1)
173:5: E303 too many blank lines (2)
189:1: E302 expected 2 blank lines, found 1
189: D205 1 blank line required between summary line and description (found 0)
189: D208 Docstring is over-indented
205:42: E701 multiple statements on one line (colon)
206:42: E701 multiple statements on one line (colon)
213:22: E261 at least two spaces before inline comment
243:1: I100 Imports statements are in the wrong order. from invenio.ext.login should be before from invenio_access.local_config
243:1: E402 module level import not at top of file
244:1: F999 redefinition of unused 'getUid' from line 189
244:1: E731 do not assign a lambda expression, use a def
247: D200 One-line docstring should fit on one line with quotes (found 2)
256:1: E302 expected 2 blank lines, found 1
256: D200 One-line docstring should fit on one line with quotes (found 3)
262:1: E302 expected 2 blank lines, found 1
262: D200 One-line docstring should fit on one line with quotes (found 3)
268:1: E302 expected 2 blank lines, found 1
268:40: E251 unexpected spaces around keyword / parameter equals
268:42: E251 unexpected spaces around keyword / parameter equals
268: D205 1 blank line required between summary line and description (found 0)
275:1: E302 expected 2 blank lines, found 1
275: D200 One-line docstring should fit on one line with quotes (found 3)
281:1: E302 expected 2 blank lines, found 1
289:1: E302 expected 2 blank lines, found 1
289: D205 1 blank line required between summary line and description (found 0)
310:1: E302 expected 2 blank lines, found 1
310: D209 Multi-line docstring closing quotes should be on a separate line
310: D205 1 blank line required between summary line and description (found 0)
323:1: E302 expected 2 blank lines, found 1
323: D400 First line should end with a period (not 't')
323: D208 Docstring is over-indented
339:1: E302 expected 2 blank lines, found 1
345:1: E302 expected 2 blank lines, found 1
355:1: E302 expected 2 blank lines, found 1
360:1: E302 expected 2 blank lines, found 1
368:1: E302 expected 2 blank lines, found 1
368: D205 1 blank line required between summary line and description (found 0)
368: D208 Docstring is over-indented
383:1: E302 expected 2 blank lines, found 1
383: D205 1 blank line required between summary line and description (found 0)
383: D208 Docstring is over-indented
398:1: E302 expected 2 blank lines, found 1
398: D209 Multi-line docstring closing quotes should be on a separate line
398: D400 First line should end with a period (not 'e')
398: D205 1 blank line required between summary line and description (found 0)
417: D202 No blank lines allowed after function docstring (found 1)
417: D400 First line should end with a period (not 'd')
417: D205 1 blank line required between summary line and description (found 0)
417: D208 Docstring is over-indented
418:9: E128 continuation line under-indented for visual indent
443:5: F841 local variable '_' is assigned to but never used
462:18: E261 at least two spaces before inline comment
464:74: E261 at least two spaces before inline comment
468:26: E261 at least two spaces before inline comment
470:26: E261 at least two spaces before inline comment
473:5: E303 too many blank lines (2)
490:23: E261 at least two spaces before inline comment
491:21: F821 undefined name 'uid'
494:1: E302 expected 2 blank lines, found 1
494: D400 First line should end with a period (not 'd')
494: D205 1 blank line required between summary line and description (found 0)
511:1: E302 expected 2 blank lines, found 1
521: D202 No blank lines allowed after function docstring (found 1)
521: D400 First line should end with a period (not 'r')
521: D205 1 blank line required between summary line and description (found 0)
537:1: E302 expected 2 blank lines, found 1
537: D202 No blank lines allowed after function docstring (found 1)
537: D400 First line should end with a period (not '1')
537: D200 One-line docstring should fit on one line with quotes (found 2)
549: D205 1 blank line required between summary line and description (found 0)
559:1: E302 expected 2 blank lines, found 1
573:1: E302 expected 2 blank lines, found 1
573: D209 Multi-line docstring closing quotes should be on a separate line
573: D400 First line should end with a period (not 'e')
573: D205 1 blank line required between summary line and description (found 0)
582:1: E302 expected 2 blank lines, found 1
582: D205 1 blank line required between summary line and description (found 0)
592:11: E127 continuation line over-indented for visual indent
597:1: E302 expected 2 blank lines, found 1
597: D209 Multi-line docstring closing quotes should be on a separate line
597: D400 First line should end with a period (not 'e')
597: D205 1 blank line required between summary line and description (found 0)
606:1: E302 expected 2 blank lines, found 1
606: D209 Multi-line docstring closing quotes should be on a separate line
606: D205 1 blank line required between summary line and description (found 0)
629:1: E302 expected 2 blank lines, found 1
629: D400 First line should end with a period (not ')')
629: D205 1 blank line required between summary line and description (found 0)
643:1: E302 expected 2 blank lines, found 1
643: D400 First line should end with a period (not ')')
643: D205 1 blank line required between summary line and description (found 0)
667:1: E302 expected 2 blank lines, found 1
667: D103 Missing docstring in public function
672:42: E261 at least two spaces before inline comment
674:1: E302 expected 2 blank lines, found 1
674: D103 Missing docstring in public function
680: D400 First line should end with a period (not 'y')
680: D205 1 blank line required between summary line and description (found 0)
717: D202 No blank lines allowed after function docstring (found 1)
740: D202 No blank lines allowed after function docstring (found 1)


Shouldn't the function getEnabledForCatalogByContextId be used here instead of getEnabledForContextByContextId?


These files seem a bit random to be included into the .gitignore. Could you be a bit more specific on why these are here, and why not add some upper folder with wildcards?


I'm sorry, I could be more verbose. To allow running pyinstaller like this: `python -m PyInstaller`


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/20079/testReport/) for this commit**


закоментований код можна видалити


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1291 of 1292 tests passed)

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


这一行会影响性能？


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1294 of 1294 tests passed)

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


done in #2955


It looks fine to me but, giving my experience in Groovy, I'm not sure if there is any values in my comment. :-)
Unit tests seems to failed.


This looks redundant?  Also I don't think that `RADICAL_PILOT_CFG` is set locally, so this also looks like a NOOP?


видалити 420 і 243 рядки


Flytta ut till configfil


Sao Đức lại commit API key lên public repo thế này :)) cho vào file config và git ignore đi


FreeBSD offer editor samples for vim and emacs at:
https://svnweb.freebsd.org/base/head/tools/tools/editing/


```selected_genes = variant_table[variant_table.Gene_ensGene.isin(targets)]``` returns a copy


I assume, this is related to https://github.com/pyinstaller/pyinstaller/issues/1430#issuecomment-136836039, where codewarrior0 is asking for more details in the logs?!


Hm, kan man inte svara på kommentar här ...
Hur mycket skall flyttas ut? Kan man få tag på ApplicationBasePath i config-fil? UseSqlite("Data source=...") kräver absolute path av nån anledning och vill att det funkar på både win och linux servern.


how many input is not limitations, but how can input be is limitations.


Puppet version: 3.8.0
I have to force the version and the ignore-dependencies... That's weird. I think that the issue is a side effect of the module dependencies. Maybe I have too many modules (34) :-( ...
```
root@puppet:/etc/puppet/environments/g10b/modules# puppet module install elasticsearch-elasticsearch -v 0.9.9 --ignore-dependencies
Notice: Preparing to install into /etc/puppet/environments/g10b/modules ...
Notice: Downloading from https://forgeapi.puppetlabs.com ...
Notice: Installing -- do not interrupt ...
/etc/puppet/environments/g10b/modules
└── elasticsearch-elasticsearch (v0.9.9)
```


General comments:
1. Pl delete HelloWorldSolution folder from test project. You added it for a testing purposes.
2. Add a exact number of passed disciplines fust right from each bar:
Babeshlo [-----------] 12
3. EACH new variable declaration starts with "var" keyword. Dont rely on js's oportunity to implicitly define and use global scope variables:
student = students[name];
4.
if (typeof student.Discipline[disciplines[i]] != 'undefined') {
                        var markText = document.createTextNode(student.Discipline[disciplines[i]].mark);
                        markCell.style.backgroundColor = color[student.Discipline[disciplines[i]].mark];
                    } else {
                        var markText = document.createTextNode('');
                        markCell.style.backgroundColor = color['empty'];
                    }
                    markCell.appendChild(markText);

this is incorrect usege of markText variable: it is defined outside the scope of its usage;
var x = {};
if () x =
else () x =
...using x...


M3113143_Ulfah Faridah


Should that be `buildsThisWeek = 1;` instead?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/20080/testReport/) for this commit**


Go to https://github.com/BensonTsai-/Javascript


what about input limitations of your input value? can it be float numbers?


What should I do next when I see this?
I should re anwser or read it only.should not?


I think we should not change os.environ underneath the running agent.  Possibly use preprocess commands on the CU to unset the respective env vars, or use a similar mechanism as popen to fully specify the env?  The first option should work with the shell spawner as it is, the second likely needs some change.  I am not in favor of the second  for other reasons though, as we can't (and should not) control what env vars the remote node needs for operation.  For mom vs. compute node that might genuinely be different...


templates


That's very weird.
IT could be that the dependencies stuff is denying the upgrade indeed for some reason.
Would be nice to find out what it could be though.
I would have thought that the version dependencies are not to strict? https://github.com/elastic/puppet-elasticsearch/blob/master/metadata.json#L11-L24
Or could it be that the ES module is referenced from somewhere else?


Are the assembly modes up to date? I uploaded 64tass 1.51.992 for OS X: http://www.popelganda.de/relaunch64.html

and I would create a new release for Relaunch64 now, if you have implemented the latest support for 64tass?

btw, have you received my mails?


that was to use qa for the test cases. Made production the default now, and
setting qa base_uri explicitly for the specs

On Thu, Sep 3, 2015 at 5:14 AM, chetanhs <notifications@github.com> wrote:

> http://www.imgcloud.io/ ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/img-cloud/ruby_gem/commit/3216943fe83a2a007fbb50a90f9ebd465a03eb0f#commitcomment-13036481>
> .
>


> I think we should not change os.environ underneath the running agent.

I agree, but the rest of the SHELL spawner already does that.


```paging(wsRequest, countTemplates)``` must be shared between lines 57 and 61


ok, so I recommend to rename ```templates``` ("pagedTemplates" for instance)


just add a  dummy field


The problem for me is the DTO is used everywhere. Internally, it's a uuid. We still have the use of db id in all the stack. It should be only ID for the external world only, IMO


That's it for the latest 0.9.9
And there is no apt dependencies into the 0.9.6, that is why puppet pulls the 0.9.6!

Thank you very much for your time.
```
Warning: Module 'puppetlabs-apt' (v2.1.1) fails to meet some dependencies:
  'elasticsearch-elasticsearch' (v0.9.9) requires 'puppetlabs-apt' (>= 1.4.0 < 2.0.0)
```


Yes.
Is there something wrong with this ?


as the WS it seems obvious to me that the search is on permission templates, and so the pagination is on templates.


ok, good point


Ahhhh okay.
Yeah, due to some issues with apt 2.x i decided not to support it yet until i have some more time to test it and ensure it all works.
I hope in a near future release i'll be able to support it.


Are you going to enable hotplugging for the driver?


Only in theory. The agent prints and checks for it. But doesn't actually use it.


it was not obvious when reading the code.


T chooseOne(std::initializer_list<T> parameters) {
   int n =  randomInRange(0, parameters.size()-1);
   return *(parameters.begin() + n;)
}


commit message: "it's" -> "its".


Use based on performance checks either
1. A regex based gsub that includes all special characters
2. Use .each.

Look at the "DRY" principle.


5.
pl use meaningful function and variables names as I told you before.
function disciplines(dataFile)  - what disciplines? may be getDictinctDiscplines?
6.
pl, use ctrl+K,D visual studio command to auto format your code.
7.
use ; symbol to finish variable declarations and function body end.
8.
box.style.width = ....
box.style.background = ....
you should incapsulata this logic into two outer functions
9.
Very important:
box.style.width = 150 * ...
NO MAGIC VALUES in code.
You should define global variable and then use it in code:
var DEFAULT_BAR_WIDTH = 150;
10.
(sortDisciplines[i][1] - sortDisciplines[0][1]) / (sortDisciplines[sortDisciplines.length - 1][1] - sortDisciplines[0][1])
this code is duplicate. Never duplicae your code;
DRY - Dont Repeat Yourself


I'll work on a proper fix.


Does this handle cases like
```javascript
obj = {"a ad" : "hello world"}


@pdsljp Why do you push commits of old repository to this repository ? We should never push commit to master branch directly. We should create pull request and merge it to master branch if there are no problems.


ok sorry, my bad. I thought you were talking about the JSON response. I agree with you, I rename


This isn't quite right.  Headers should be a single Hash *OR* an Array of Hash.


Here you go:
https://github.com/elm-lang/elm-compiler/pull/1044


a resource with an empty name looks weird. You can support this, but then write a test to demonstrate it's not a bug.


fine for me


username_exists returns false not null


I'm bad - that was a big merge a while back. :/ +2


Thanks!


bound-checking is missing.


It passes now. https://github.com/commercialhaskell/stack/commit/5f3ffe5ea8b4d8d1f4d1e4d15bee8b481ec6a8d4


Первые строки одинаковые, отличия только: `href="${url}"` и `href="<c:out value="${url}"`


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/20081/testReport/) for this commit**


Yes, if you don't indefinitely keep references to past points in the stream, then the past stream will be GC'ed (very much like with CML multicast).  So, yes, you can have long running, even indefinitely running, stream consumers (to avoid confusion with Rx style semantics, I would avoid calling them subscribers).


@yordis Sure. Could you open an issue with more details? Thanks.


Ясно теперь, переделаю.


Who is meant to be noticing if this is taking too long?


Hi
I’m afraid I cannot help with the coding of the module issue. I vote for stability over multi-module support. I think we need to get this out, even with a reduced functionality.

From: Joel Andersson [mailto:notifications@github.com]
Sent: 04 September 2015 08:47
To: jaeandersson/swig <swig@noreply.github.com>
Cc: Thielemans, Kris <k.thielemans@ucl.ac.uk>
Subject: Re: [swig] [MATLAB] Replaced swigInd pointer to raw pointer (f93302e)

@jgillis<https://github.com/jgillis> @KrisThielemans<https://github.com/KrisThielemans> @dcb210<https://github.com/dcb210> Apparently this commit did little to resolve the "modules" issue. And since it could potentially lead to more nasty failures (segfaults), should we revert it?

—
Reply to this email directly or view it on GitHub<https://github.com/jaeandersson/swig/commit/f93302e9045d436c0a83d348497d20920875999c#commitcomment-13063865>.


This didn't work for me, if it doesn't work for you let me know and I will pull request what I did here, was tricky, I used 3 different settings commits as reference


Wo hast du denn die Syntax mit dem Symbol her?
Ich hatte eher sowas wie

    link_to "Rangliste", rankings_path

im Sinn. Und für Home vielleicht sogar besser

    link_to "Home", root_path


Maybe my usage is wrong bu I've the following error when using you comit:
java.io.FileNotFoundException: C:\sources\git\ul-trader\ul-trader-desk-api\build\tmp\nugetSpec\ul-trader-api.nuspec (The system cannot find the file specified)


ok for me on the principle of this patch.


`m_texture` is NULL first time around this loop, so I was wondering if this line should be using `texture` to create the `TextureObject` - having said that, I've not noticed a problem with the viewer.


do you need parentheses?


Maybe we should remove the chat, auto-ipv6 and pberg wirzard apps. "Nobody" is using the apps and they create unnecessary work.


its


To clarify: Module support is possible with or without use of raw pointers. An approach would be to add a central mex file used by all modules (just like we have the SwigRef class as a common base class for all modules). Anyway, I'll revert this.


Salut,
 Pense à cliquer sur le bouton "create pull request", sinon on ne peut pas inclure ta modification :-)


outdate comment


:ok:


remove outdate comment required


You did it twice just do it once


Loading a textTemplate from the fs is very easy:
````java
Text myRawTemplate = Text.json().from(myString);
Map<String, Object> specialFormatters = Maps.newMap();
specialFormatters.put("displayName", Texts.placeholder(TextArgs.displayName("player")));
specialFormatters.put("hostileCheck", Texts.placeholder(MyArenaTextArgs.areHostile("player", "other")));
specialFormatters.put("location", Texts.placeholder(TextArgs.function("player", player -> myPlugin.locate(player))));
Text myTemplate = Texts.format(myRawTemplate, specialFormatters);
````
The above steps are necessary because you cannot create TextArgs from the json.
Theoretically you could even do that, but I don't want to include a compiler step there and also don't want to use strings to somehow reference to a TextArg class/constructor/factory.
In almost all cases the useful formatters are very specific for a certain message, that's why its better to apply the formatters to that template directly after loading the template from the FS.

AFAIK gratimax's proposal has the same issue.

**PS: Dark magic**
````java
Function<String,?> function = (key) -> {
if (key.contains("TextArgument") {
    // key = "DisplayNameTextArgument{ref=player}" or "MyArenaDistanceTextArgument{self=player,other=other,unit=m,limit=15}"
    TextArgument arg = recreateTextArgFromToStringResults(key);
    // arg.toString() == key
    return Texts.placeholder(arg);
} else {
    return null;
};
template = Texts.format(rawTemplate, function);
````
If you use this way you only have to make sure that your Formatter functions have a good `toString()` implementation that serves enough values to restore it from that String.


r+


@delcypher In which sense are they ubuntu specific? In general it's already quite Ubuntu/Debian centric.
I like to suggest Ubuntu as a distribution for most people. It lowers the entrance burden, i.e. to explore KLEE (e.g. Docker images are based on Ubuntu).
For other distributions, they are behaving similar and/or users are more advanced linux users and typically able to reproduce the intended behavior on their sytems.


LGTM otherwise


'e' is guaranteed to be de-referenceable at this point? Look at what JavaScript is making me do. I can't say 'non-null' anymore.


likewise


See #389.


did you actually do this one? Lets link up on the exercises and come with questions then try redoing this exercise.


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/bb07f5097d9b0ddd9bbda846bebd3af8f83c8474


merging MonsieurNicolas/stellar-core/offerPrice = bb07f509 into auto


MonsieurNicolas/stellar-core/offerPrice = bb07f509 merged ok, testing candidate = a2edea93


I half wonder if this should be a .ui file now. Oh well, don't give it much mind.


Is this the best way to check to see if an option is not existent?


Frankly I'm finding that whole area of the code fairly overcomplicated. Have you run across this as a bug in an actual use case? If so, then yes, you're probably right :)


This change seems not to have made it to the version published npm. Relevant line still says:

`    _middlewares = middlewares;` on line 41 of the transformed source.


Typo


instructions say use print_r() :)


93651760-RowPerObjectReport (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1828 of 1828 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)


Please convert spaces back to tabs and set up editor for using tabs instead of spaces.


Add enyo DCO to commit and/or pull request


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2320 of 2337 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)


@CodeFlowed As you said, maybe adding a `before_action :this_game, only: :show` and then doing
```def this_game
@game ||= Game.find(select_pc.game_id)```
Just a thought...


you don't need this check... since you are removing the cards from the deck when you do array_rand the next loop you will only get cards that are still in the deck


That's what I usually do.


This breaks 9858 reset.


by "provided sample" do you mean the seed project? or just the code sample below?


Add a comment that this is for vertical centering, if there's not an easy way to center this right now.


The Windows vetting was something I was leaving to interested parties to do... so a few days of slow from you is no problem, as people haven't really been jumping on it so far.  But it would be nice to be able to report the serial stuff is in working order!  Keep me posted and let me know if there's anything I can do to help, I hope the comments are helpful and if they're not feel free to update any that aren't.


You already have the results of createDeck() in $deck pass that into count()


Brady, Use xlt for these?


86139842-AddAutoSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (765 of 765 tests passed)

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


changed the excaping


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/78849037


Should they be marked as deprecated since they're not used in the client anymore?


You should use $num_players instead of 4 here


Bakit ka sa master nagpush?!?! Tsk tsk.


Needs tests.


Should those methods be also kept for backward compatibility?


even though this works, nice job btw, the instructions say "until they are out of cards" so you should probably have unset the cards selected from each players hand and looped until they were all out of cards... just pointing that out :)


Confirmed on my test site. Thanks.


Not sure what you are wanting here


(Addressed in https://github.com/magfest/ubersystem-puppet/pull/54/)


@anseljh please don't feel at all like you should track my preferred coding style of the moment. I have my reasons for using it, but freely admit it's unorthodox. Many folks I highly respect have decided to render the whole coding style thing a nonissue with [standard](https://www.npmjs.com/package/standard).


There was a lot of copying and pasting going on here, so it was a lot easier to keep your style!


FYI: the ^ or ~ have no effect before 1.0.0 because 0.x versions are considered unstable yet and can have breaking changes at any time


Maybe it's just too late in the day but this doesn't read right for me... won't this try to dereference ``ip.Arn`` if and only if ``ip`` is ``nil``? I think maybe you wanted to say ``ip == nil || ip.Arn == nil`` here, so it'd short-circuit on that first test and not try to evaluate the second expression.


wouldn't such suit better in a branch `gh-pages` or even better a dedicated repo `shaarli.gihub.io` ?


First, thank you for your time to explain this.
I didn't know that you could replace a simple placeholder with an enhanced `TextArg`.
Is it correct that if the `Function` returns null, then the placeholder isn't replaced? I don't see this anywhere in Javadocs.

After I understood this, I would definitely want to get this in.
But I still struggle to understand in which way this PR differs from gratimax's proposal. For me it seems kinda like the same idea. :(


As my consideration, resource/assets download should be core function. Extended function might be more "regarding-less" to main function such as an alternative GUI or modpack management.


The getVisibleScrollerText was important to this test as there was a bug where the content was correct but the scroller was not scrolling to the correct spot.


What If I told you that if you put `'use babel'` as the first line of a javascript file, atom handles the transpilation and all


I would still like `$domain` to be on line 76 so you can see what lists are being skipped, so:

```
echo "Skipping $domain list because it does not have any new entries..."
```

Other than that, all the rest of your commits look nice.  I'm just going to do some more testing this weekend before merging.


Definitely -migration to ReadTheDocs (RTFD) is planned in #312


while what you have works you should declare properties on this class for width and height, such as:

```
public $width;

public $height;
```


Add headers


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 518](https://teamcity.labkey.org/viewLog.html?buildId=270319&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 28, passed: 9 Build time: 00:23:34
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 18 more failed tests, see build details
```


when using foreach on an array that contains object the "as" or value can be the object, so:

```
 foreach($this->items as $item){
// then $item is the same object you passed into the addItem method
```


this var_dump shouldn't be here. also you are not returning the sum once the foreach is done running.


not sure what you are trying to do here... lets talk about this when we meet.


Should we worry about `304 not modified`?


since $name and $price are public properties of the class you don't need a function to get the name or price. you can just call them directly like $class->name or $class->price


Indicative of possible over-enthusiasm - will review within 24 hours whether v14 shortcuts are ripe for removal :)


lets talk about this. maybe going though the exercises together will help clear up this challenge.


oofs :( rookie mistake, sorry. If @domenic has not already fixed it, I will. (I blame @annevk for asking me to make it one huge comment, but more than that I blame myself for **not actually having tested it by running the HTML build after I made the change**.

Lesson learned: No change is too small that I can skip the rule that I (we) must run the build after making the change and at the very least make sure the build doesn’t fail.

Again, sorry. Breaking the build is nothing to brush off, and I will make sure I don’t do it again.


I guess so. The settings button just opens the file. I think is kind of nice that no time is wasted on a dialog just a simple confit file. And perhaps get did not want a full js module. How ami not surprised that MS extended Json. At least it's not xml. The vireo Json parser allows for comments too, but I'll try to confine comments to *.via files.


Thanks and consider me admonished. I should be forced to wear a dunce cap for a week…


great job, ill pull this in.


Who made @amoghbl1 : The admin :D


1- Sua regra daria para dar merge; está atualizada. Mas cadê o pull request?

2- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.


93651760-RowPerObjectReport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1828 of 1828 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)


now we have edit the code and worked on dynamic data entry in the xml file on run time


I think they are compatible. The charge model they are using is also RESP. Even they are using a different method (scanning the PES) to obtain the bond and angle parameters while MCPB.py uses Seminario method to obtain these parameters, the parameters based on two different methods could be comparable.

> On Sep 3, 2015, at 2:27 PM, lizhe415 <notifications@github.com> wrote:
>
>
> > On Sep 2, 2015, at 10:29 PM, pengfeili1 <notifications@github.com> wrote:
> >
> > Hi, a commit to solve this problem has been submitted. Please test the new code to see whether it works.
> >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/Amber-MD/pymsmt/commit/ad8f3fb65aebc22ed068d64cfccb003c2ac9b038#commitcomment-13037780>.
> >
> Thank you. Now it works.
> But I have a question about the parameter, Amber compatible parameters for heme were reported in the literate (J Comput Chem 2012, 33 (2), 119-133.) where non-bonded terms of (1.3, 0.01) were reported for Fe4+ in the supporting information. Are these parameters compatible with your tools?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Amber-MD/pymsmt/commit/ad8f3fb65aebc22ed068d64cfccb003c2ac9b038#commitcomment-13054554>.
>


This means that your precision is +- 1 million, is that correct ?


missing a check for utah I am guessing


This commit addresses issue #233


This commit addresses Issue #234


actually, lets remove this. There is a jira we opened today where snapshot plugins will break if the cdap master restarts.


@cybersiddhu this is another way to do it


At least for the purpose of config files this has the endorsement Mr JSON. Perhaps because it's not generic data.
http://stackoverflow.com/questions/244777/can-i-comment-a-json-file
But to GitHub it just JSON so the complaints make sense. Should the file suffix be jsini ;)


1- Falta um merge. O upstream andou. Corra atrás e atualize.

2- Coloque seu nome  globalmente no config do servidor camelo (estava fulano de tal aqui no commit anterior; se já atualizou, faça commit novamente com autora correta).

3- Você colocou diversos caracteres estranhos no arquivo texto das regras (saída do diff). Limpe e faça novo commit.


这一行可以用 gulp-autoprefixer 自动添加


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

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


Você adicionou vários caracteres estranhos no arquivo texto (a saída do comando diff). Limpe-o e faça novo commit.


use all caps ERROR_SCHEMA for consistent style


You checked in with failing test!


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


 Seu pull request está ok, mas só posso aceitar depois de você corrigir:

1- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.


OK! Pull request aceito.


1- Falta um merge. O upstream andou. Corra atrás e atualize.

2- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.


weird white space shuffle going on here.


1- Falta um merge. O upstream andou. Corra atrás e atualize.

2- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.

3- Faça o pedido de pull request


:+1: great, should've been there from the start


1- Falta um merge. O upstream andou. Corra atrás e atualize.

2- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.

3- Você adicionou vários caracteres estranhos no arquivo texto (saída do diff). Limpe-o e reescreva sua regra.


No need for a new function, just add the offset as an arg to get_node and add the ioctl to set the offset. The existing call of get_node would just pass an offset of 0.


You should be able to probe image_path instead, no need to create a loop dev just to read the partition table.


so, when doing `ipfs add -r ` on a big dir, i may know i dont want to add the pipe. maybe we want an:

```
--ignore-unsupported-files
```
flag that just keeps going. (future PR)


1- Falta um merge. O upstream andou. Corra atrás e atualize.

2- Coloque seu nome tanto no github quanto globalmente no config do servidor camelo.

3- Você colocou caracteres estranhos no arquivo (saída do diff). Limpe-o e reenvie.


You need to put parens around if (buf) now that it is more then one line. WaitForAttention passes NULL for the buffer since it only wants to know when an attention report occurs and doesn't care about its contents.


Has this been tested with other mobs?


I left it out because it prints on the same line as:
echo -n "Getting $domain list... "


When I tried to do this I ended up getting a bunch of errors with things like `connect`...


@erics273, since @anchavez has submitted his work can we go though the challenge so he will understand how to do it? or should we go through exercises and I let him do it on his own?


this should be removed too


we know all the transform ids at this point.  Can populate map with those ids as keys and empty Lists as values to simplify logic later on.


![2015-09-01 13 44 34](https://cloud.githubusercontent.com/assets/14066682/9697025/72f9adac-53b8-11e5-9468-bbc1d50ba1fc.png)
GitHubでのコメントwithスクリーンショット


I have no idea what this line is for. It's outside functions and seems to have no purpose.


don't we need to clear transformIdToErrorRecords?


Is this right, removing from the front of the list? or should it remove from the back (list.size()-1)


we should log as a warning once, then silently ignore after that.


何だかなあ〜


XD


What is the behavior of this?  From the one-line description it sounds like it might be intended to prevent breaks that are normal and desirable, such as:

a) The sequence `[word][space][emdash][space][word]` is commonly used to allow a break before or after an em dash by default, in addition to adding some space around the dash.  In this example the em dash doesn't *immediately* follow a word --- it follows a space that follows a word.

b) Depending on usage, number sequences can be broken around an em dash, hyphen, or `\%` hyphenation, but not (normally) around an en dash or a minus sign.


rx.rORc == r0.rORc is all that's necessary. The request type is rORc not reqType.


still need to use a better row key.  Don't use string representations long and uuid, use their bytes directly.


Thanks!


Also with npm-shrinkwrap.json in place, can you add the ^caret to all the versions? I care about this for optimizing bundle size.


shouldn't have to check this since we know transform ids beforehand.


if you change transformResponse to return `Map<String, Collection<InvalidEntry>>`, can just call addAll at this point


I think it would be great if you worked together to help him understand how
to solve these challenges. We can catch up on this Monday. I really
appreciate you taking the time to coach Anthony. Im am sure you comments
would help others as well.

On Fri, Sep 4, 2015, 9:24 PM Joseph K Baker <notifications@github.com>
wrote:

> @erics273 <https://github.com/erics273>, since @anchavez
> <https://github.com/anchavez> has submitted his work can we go though the
> challenge so he will understand how to do it? or should we go through
> exercises and I let him do it on his own?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Code2Hire/challenges-student-work/commit/a369dfa4caf98f1d883d2734ee72a8a313ce5deb#commitcomment-13080496>
> .
>


if we'll need to enable this we'll need to copy csc.exe.config to where native Roslyn binaries are; otherwise we don't need csc.exe.[il]config files


Thanks for this!


That precision is not related to the method itself. It is just for this example, which if noise free. The only attained error is the machine number precision, and the function to optimise is convex, so actually the expected error should be much more smaller.


should be already fixed


Perhaps try sticking in an `#include <windows.h>` before line 46?


I think issue was about md5 and RS2 optimizations, fixed this one, gotta commit soon


not all of them have configured error datasets right?


don't need this file


I learn on web-teaching site...but already don't know how to begin a one sep.../.\


Тогда уж для наглядности `flex: 0 0 auto;` стоит писать как `flex: none;`.
Алсо, в `#de-panel-buttons` (и вообще внутри флекс-контейнеров) вместо `display: inline-flex;` можно писать проще `display: flex;`, потому как все флекс-айтемы, даже с явно прописанным инлайн-лэйаутом, автоматом превращаются в блоки.


main / my page 통합


I wouldn't usually support 1.8, because it is indeed ancient. I'll make an exception for this gem (for now), because the changes were trivial: 1eca0d9 – but I encourage you to update to 2.x as soon as you can ;)


Что-то Избранное теперь вообще хаотично открывается - при открытии нового треда в той же/другой вкладке, при переходе по страницам доски (до этого оно ессно закрыто).


oh shit..   extra `:`


93651760-RowPerObjectReport (JasonGlazer) - 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)


:heart_eyes_cat: :heart_eyes_cat: :heart_eyes_cat: :heart_eyes_cat:


 No. Some servers will use their own launchers. They only need to launch
existing configured minecraft. So they don't need to download
resources,assets. This is a optional feature. If you need this, just add
this to your classpath. Else if you don't need, just exclude this, and your
jar will get smaller.

2015-09-05 8:42 GMT+08:00 Derek.CHAN <notifications@github.com>:

> As my consideration, resource/assets download should be core function.
> Extended function might be more "regarding-less" to main function such as
> an alternative GUI or modpack management.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/to2mbn/JMCCC/commit/929f8db6e080e63fb03cd007a9816eac62af8380#commitcomment-13080262>
> .
>


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


遅れました、マージしちゃってください！


can we call this method "toSmiles" like in OCL and to reflect the name from RDKit ?


Your code is needed?
```reddift``` uses ```KeychainAccess``` as not Framework but just source code.
So, I think that I don't have to import ```KeychainAccess```.


Can we call it "toMolfile" as we are never adding more than one molecule to the sdf with this method


No, that isn't the case. Jack doesn't accuse anybody. He only states that Matthew and Peter are both lying. Matthew on the other hand says that he and Carl didn't do it. I see this as an indirect accusement for Peter, Jack and Arnold. You could argue that you should say that Jack, Carl and Arnold indirectly accuses everyone, but I think this is not correct.

Hereby the statements so you don't have to switch back and forth between webpages:
- Matthew: Carl didn’t do it, and neither did I.
- Peter: It was Matthew or it was Jack.
- Jack: Matthew and Peter are both lying.
- Arnold: Matthew or Peter is speaking the truth, but not both.
- Carl: What Arnold says is not true.


BTW, should the issue title read "fix >=4GiB files"?


This fixed the issue #128


I think your solution is more elegant. I will change it.


This should read "SFML depends on a few external libraries on Mac OS X.". Patch will follow soon.


Unfortunately WorldGuard doesn't allow any other way to fetch their instance. :(


We need to do this so we can show a message. Unless ```com.massivecraft.factions.Perm``` allows wildcard sub-nodes?


I understand what you mean and I agree, but I use the function names that are mentioned in the assignment.


this is just a manual merge becaused i messed up the merge conflict while merging my PR #419


This is the latest commit I can build from a clean repo checkout.


Have a look at the ``unittest.skipIf`` decorator to get rid of conditional checks at the global level.

```
@skipIf(not bempp.HAVE_DOLFIN)
class TestP1Coupling(TestCase):
     ...
```


You mean we should still check for padding == 4? In that case we can just do this instead of calling check_pkcs_pad():
```
	if (((unsigned int)out)[11] != 0x04040404)
```


can we make this plugin in such a way that users come and light a candle for someone and the number count of candles lit by the users show below the image or post.


this message should go to error output too (yes, old code was incorrect)


Sure. It suspends pulseaudio temporarily and allows the program it starts (jackd) to use the ALSA device, resuming it when the child process (e.g. jackd) terminates (or did you mean in the PR?)

FWIW, I can't get sonic-pi working. I can use mplayer with jackd fine, and can see that in qjackctl > Connect , but sonic-pi doesn't show up as a "readable client" when I run it. I'm using the latest check-in from git, so maybe I should try a release like v2.6.0 instead. I'm using Ubuntu 14.04 LTS, by the way.


going to integrate this commit into devel:

git cherry-pick 8b2efc0086f0c370607d2ceada379e3738a76839


Yes, "padding == 4" was my way of reducing the false positives, and it seems to work in practice.

I am still worried about false negatives (missed cracks) but I haven't seen one, and I don't know of a better method yet.


When negating a boolean, we put spaces between the braces and the "¨!" for easier recognition:
if (!event..... --> if ( ! event.....


コミット


Been upped to 14 in dev-head


And this might be personal preference, but the field name should probably be mplayer instead of player.
Otherwise people might mistake it for a Player object.


@Fyrd Eh?


Agreed - I did mean to do this and then forgot!


OK, but that would break non-intel. I'm committing this
```c
	/* possible bug here, is this assumption (pad of 4) always valid? */
	if (out[47] != 4 || check_pkcs_pad(out, CTLEN, 8) < 0)
		return -1;
```


Added now


On dolistore, there is a test to check that a module does not contains files of dolibarr core (like htdocs/core/modules/). However, we introduce this very late and i am not sure it works correctly. So i am nearly sure we still accepted external modules with contant that is not "only the dir of module" but overwriting also files of core.
So the test on if type === main is a good think because we are sure that modules, at least, does not use the /core/includes intead of /module/core/includes. But currently, it may borke compatibility with bad modules for such cases. And a user should not experience compatibility break when updating to a minor version.

So I would prefer you introduce this into develop branch only.


Thanks for watching my back!


Ah, okay. Wasn't in the Style Guide - but i'll remember for the future.


missing redirect to stderr


There will still be one FP in every 4 billion candidates though, that's a lot.


I understand why you use `true == booleanVariable` instead of `booleanVariable == true`. Bug protection over readability. But I still don't understand why you compare booleans in first place: `if (true == booleanVariable)` does exactly the same as `if (booleanVariable)`. Everyone I know says that using comparison operators with booleans is a bad practice.


Thanks :wink:


What does the `element` variable mean? What is it for?


Good stuff; the new movement works really well


I had a separate local master that I then remastered into a merge with master.


Yeah, but no decision.. This one is probably easier than the popup menu one though, this one is not as busy.


and please line break to 80 character limit


revert this commit please )


So 'secretly' even an improvement for (previously) unknown densities, that now also get automatically the highest DPI package, if I understand correctly?


@nigelpain

Have moved it to the background section.

So general rule of thumb, if first step appears in more than one scenario, move them to background section? Only applicable to first step?


Dope bruh


Cool :+1:

I spent a lot of days in trying to reduce the false positives (and at the end, I had to come up with this crappy logic). This is why I remember this so clearly :-)

Also see https://github.com/juuso/keychaindump/blob/master/keychaindump.c (which doesn't use this "padding == 4" trick). This trick makes sense to me because the size of the underlying data blob is fixed (IIRC). So the encryption of this data blob always uses a padding of 4 bytes. It would be awesome if someone else could confirm (or debunk) this!


Done - makes sense


You should change "light" to "dark"!


Oops - have removed


Yes, looking at the code I found out that unknown density would break line 310: *if [ "$d" -ge "$density" ]; then*, since *unknown* is certainly not a number.


This is a bug: the function may only return 1 if a codec is found. The `return 1;` statement should be inside the `if {}` clause above.


I don't really like the `0x6`; can you replace it with a named constant?


Perfect! I'll merge it to the master.


Agreed - can see the point of doing that.

Have added an extra step to select the returned property.


for top1m we care only for `CAPATH` (which is much faster than `CAFILE`), so if `CAPATH` is defined, the whole `CACERTS` related logic can be skipped

while `--capath` wasn't supported, `CACERTS` env variable with exact same meaning was supported - I'm not so sure if we want to drop that (line 57 in new causes that)

but we can safely check if both `CACERTS` and `CAPATH` are set and abort if they are

you're also missing addition of description of `--capath` to "usage"


Agreed. Have updated to read

an invalid Title Number error message is displayed to the user


I think we should add WebStorm as well:
https://www.jetbrains.com/webstorm/help/using-javascript-code-quality-tools.html#installESLint
Because according to: http://angularjs.blogspot.de/2015/09/angular-2-survey-results.html Webstorm is one of the most used or the most used Editor.


test


doing a
```
if [[ -n $CAPATH && -n $CAFILE ]]; then
    echo "Both directory and file with CA certificates specified" 1>&2
    exit 1
fi
```
after all options are parsed (after `esac`) would be cleaner and reduce code duplication


as I said in e35a615, not so sure if we want to drop it...


I just had a quick chat with Cayorion. We never use the OP status for decition making. He stated that you should rather check for a permission (or in this case, admin mode is sufficient) as it is the result, rather then checking the OP status is a cause.


r+


Yep, thanks ... good to know  !  :)


Bit of a comedy of errors on my initial commit. But this is what I meant to do:

1. Made width/height/thickness conditionally appear, so if one's missing it's no big deal.

2. For that same reason, I don't think we can just dump the whole format on a translation key. If we did a translation string like `{width} x {height} x {thickness}`, the translator would have no way of accounting for situations where a dimension doesn't exist. For instance, if no thickness data was available, it would read `12cm x 15cm x`.

Ok like this?


Ok, we can discuss that this afternoon. In my opinion, if matthew says that carl and he did not do it, and Peter says that he's lying, then Peter is at least saying that one of them did it.


可能是地区问题、网络运营商或是浏览器问题，安徽联通正常访问

jy02534655 <notifications@github.com>于2015年9月5日周六 下午5:17写道：

> 申请注册一个Google App Engine账号
> https://appengine.google.com时，https://console.developers.google.com/这个网址打不开，求解决
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/racaljk/hosts/commit/809173c3a9e78f64770bc8a5b8eccf4ee3a6e110#commitcomment-13081645>
> .
>
--
Andy


Sorry, I think maybe I misunderstand something: the modifications showed in this page, only work if the site is hosted in GitHub, and **do not** use custom domain.


I'll have a look. Might take some time, it's all spare time work

;-) Matthijs

> On 5 sep. 2015, at 12:14, debasis1 <notifications@github.com> wrote:
>
> can we make this plugin in such a way that users come and light a candle for someone and the number count of candles lit by the users show below the image or post.
>
> —
> Reply to this email directly or view it on GitHub.
>


Solves #1  and #2


this fixes https://github.com/ocaml/ocaml.org/issues/698


Merge these classes into one if they're synonymous.


Hi there ! http://hpics.li/306555a


Honestly I didn't know I had `.` in my path, nice catch =D


Damn @h2o64 thats nice!! Ill be waiting for your sources :D


Yes they should be added, I agree.
I thought I did it , but I cannot find the text anymore.  May be I made a mistake and lost the doc on this :(


@jlec Isn't this somehow a circular dependency? `ipython[notebook]` pulls in `notebook` which required `ipykernel`: https://github.com/gentoo-science/sci/blob/master/dev-python/notebook/notebook-9999.ebuild#L35


The change occurs within a C++ template.  How could it effect C?

> On Sep 5, 2015, at 4:51 AM, Joel Andersson <notifications@github.com> wrote:
>
> Apart from involving changes to other language modules, this is breaking if you generating an interface to C (not C++) code.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dcb210/swig/commit/0a44eb80dba29a94acdb22fca6d8b0dbc286b240#commitcomment-13081584>.
>


Typo of course! Thanks for catching this.


Add SORGQR, CUNGQR, DORGQR, ZUNGQR to Generate Q from QR factorization


It's easier to understand this way, but I'm not very fan of hard-coded string to repeat. If you want to do it this way, please use constants like:
```VISIT, MERGE, HANDLE_RECURSION = 0, 1, 2```

And then re-use this variables.


Is this going to show on the site anywhere?


isolate these lines in breakpts#GetLocalizedStrings


Actually no I didn't do it.
Could you add some doc about this plus a simple example.
I will review afterwards


Do we need this? The 1.1 spec is from 2005, I would assume that all usable implementations out there are 1.1.


you have made change but   need to change colour for better look such as change bright colour.


Octree (DeadParrot) - 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)


It helps with error but when create or try auth via api auth token is null:
"authentication_token":null


@graemerocher Please enable issue reporting in this repo as Jira is read only


I tried to be consistent with Integer.MAX_VALUE, Byte.MAX_VALUE etc. but you used these already. So I decided to take a similar name. There is also the use case that UByte, UShort etc. were used in the same class with static imports, which is only valid with the longer names.


I don't know much python either :-)


IMO what @amirian28 has done is just frivolous. 6 lines code don't need to create a new file, right? Normally custom cmake files are:

* Handy macros.
* FindPkg.cmake (Thanks to FindPkgConfig, it's pretty simple now)

But, the most important principle is, don't break stuffs.

```install(SCRIPT)``` invokes scripts as a independent file (`cmake -P`), so `${PROJECT_SOURCE_DIR}` is void. I kept hitting failure of finding /conf/zmap.conf which certainly doesn't exist. :joy:


Of course, stupid of me. Nevermind C vs. C++. In any case, we should probably leave the other languages alone. Or make a separate pull request to SWIG's master branch.


r+


What is `increasedElement`?


Consistent would be to use the same name in all four types, as this is already the case for `Integer.MAX_VALUE` and `Byte.MAX_VALUE` :)

Anyway, I'll think of a name and rename them after a merge...


Works if you change:

pid, proc = parts[6].lstrip(":").split('/')

into

pid, proc = parts[6].rstrip(":").split('/')

==> Colon needs to be removed from the end of the string, not the beginning.


không cần "value": 'NaN'


Hi osmcers, I believe this should be
` chown -R $OWNER /usr/share/kodi/addons/resource.language.*`
(Note the missing 'u' in the original).


Github said:
 >ProTip! Notify someone on an issue with a mention, like: @jojoe77777.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 519](https://teamcity.labkey.org/viewLog.html?buildId=270419&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 4 Build time: 00:30:44
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 23 more failed tests, see build details
```


@CeytiX you should leave a line comment above *if statements* like this so it's easier to understand *why* and *what* you did.


Maybe a Overflow/UnderflowException would be a better choice. But I think throwing an Exception is a better decision than a silent overflow. See also the Java Puzzlers book. C# decided for reporting the overflow/underflow, there you need an additional language construct for allowing silent overflow/underflow. Maybe we could add two additional methods add/subtractWithWrapAround to make the behavior clear.
I need the add and subtract methods for handling binary protocols and there is overfow and underflow an error.


> How was this auto-generated?
via the repository-client setversion command


@CeytiX you should name *bool* varaibles like `isCritical`, `isOpen`, `isDone`, so it's easier to understand the code while reading *if statements*.


Good points. Let's keep the additional methods as an idea for the future. I suspect that right now we don't need them, really.


I'm in a similar situation :/


@CeytiX what do you mean by this line?


@CeytiX why do you put `{ }` brackets instead of `( )`?


This commit needs a bit lazyness, but right now, it's fine to just be accurate


the `element` map contains 4 variables= "fire", "air", "earth" and "water"
they allow the  player to have bonuses or maluses against other players


@CeytiX I think it would be better to call this method `updatePlayerStats()` instead of `update()`.


see https://github.com/epeli/underscore.string/pull/448


Should the exception also point the user to thread->wait()?


the player wins 1 armor and 1 magic resistance 2 out of 3 levels


to respect c++11 std


increasedElement = base element + bonus element (for each string in map)


I was’t sure where to submit the pull request.  The change needs to occur in all of the languages.  My thought was to submit it to your Matlab branch, since I need to use it for now for Matlab support, so that it can be incorporated into the master SWIG branch when that occurs.  Of course, I’m assuming that your goal is to submit your branch to SWIG master when it’s ready.

> On Sep 5, 2015, at 7:20 AM, Joel Andersson <notifications@github.com> wrote:
>
> Of course, stupid of me. Nevermind C vs. C++. In any case, we should probably leave the other languages alone. Or make a separate pull request to SWIG's master branch.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dcb210/swig/commit/0a44eb80dba29a94acdb22fca6d8b0dbc286b240#commitcomment-13081987>.
>


can be merged with the previous case


As of method name it's not supposed to be used with anything except bool. Either defaulting to `False` or accepting `None` as `False` is flawed since it may have opposite meaning by default. If you want it to support `None` `none_value=None` can be added.


thanks, i think i will change


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 520](https://teamcity.labkey.org/viewLog.html?buildId=270384&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:23:42
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 20 more failed tests, see build details
```


i'm about to do it so


I'm wondering about the "should have" in this comment. Is this a TODO?


@CeytiX wow, I would recommend putting each parameter on separate line.
``` c++
Axe::Axe
(
    int range,
    std::string element,
    ...
)
{
    ...
}
```


i won't create another updating function so it's not really necessary


Nope, it's fine on windows in general.


i never thinked to do it, thank you


Removed typo from list


fixed!


@CeytiX in parameters I would use a reference: `std::string&` instead of `std::string`.


the size of the string is between 3 and 5 so it's not very usefull but I remember it


Good catch. I just pushed a commit to fix this. Thanks!


The return type changed to `bool`, so the code should `return true;` here. (After rebasing onto the new master.)


It seems this is the only line that's actually changed but your commit seems to have picked up other things due git not being very smart. Would you mind cleaning up the commit.


Could you please add a line why you reverted the commit? Otherwise it looks odd for the casual reader.


Agreed, will implement as provided.

On Sat, Sep 5, 2015 at 3:46 AM, Hubert Kario <notifications@github.com>
wrote:

> doing a
>
> if [[ -n $CAPATH && -n $CAFILE ]]; then
>     echo "Both directory and file with CA certificates specified" 1>&2
>     exit 1
> fi
>
> after all options are parsed (after esac) would be cleaner and reduce
> code duplication
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/floatingatoll/cipherscan/commit/37871fb46946585f921a7dd574ed9f69a95fb5e4#commitcomment-13081881>
> .
>


@CeytiX what does this line mean?


good


getAvaibleWeapon() returns 2 if the player is avaible to equip weapon, 1 if he could be avaible to equip weapon (but not yet) and 0 if he cans not


@christianalfoni Hi, is the right place to change the page title in an action with cerebral-router ?


Octree (DeadParrot) - 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-81.73%25-red.svg)


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


See it. It conflicts with Source Highlight Feature, and now I disable it temp.


This should still be 300 by default.


@ftonato excelente ponto!

O que precisamos então é melhorar o README da master, pois acho que a pessoa fica perdida lá kkkkk (ao menos quem não conhece que o conteúdo está no gh-pages)


bbb


I decided to remove the zeroing of CACERTS="" that I added, and the
preexisting zeroing of CAPATH="" that you added (77671137), so that both
can be overridden either with --cafile or --capath or CACERTS or CAPATH.
Which supports all preexisting behavior and even adds CAPATH support so it
behaves consistently.

I interpreted "--capath to usage" to read "--cafile to usage", which is
indeed correct. I'll fix.

 - R.

On Sat, Sep 5, 2015 at 3:43 AM, Hubert Kario <notifications@github.com>
wrote:

> for top1m we care only for CAPATH (which is much faster than CAFILE), so
> if CAPATH is defined, the whole CACERTS related logic can be skipped
>
> while --capath wasn't supported, CACERTS env variable with exact same
> meaning was supported - I'm not so sure if we want to drop that (line 57 in
> new causes that)
>
> but we can safely check if both CACERTS and CAPATH are set and abort if
> they are
>
> you're also missing addition of description of --capath to "usage"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/floatingatoll/cipherscan/commit/e35a6155bc6913793d4e5a6ac168d5ab2c952471#commitcomment-13081872>
> .
>


I wasn't too clear. The problem is that youtube-dl won't fail but some programs using the youtube_dl module can fail, for example:
```python
import youtube_dl

params = {
    'external_downloader': 'aria2c',
}
url = 'test:youtube'
ydl = youtube_dl.YoutubeDL(params)
ydl.extract_info(url)
```

There are some options:
* Add a `default` parameter:
```
def cli_bool_option(params, command_option, param, true_value='true', false_value='false', separator=None, default=False):
    param = params.get(param, default)
```
* In `YoutubeDL.__init__` call `params.setdefault(key, default_value)` for all the parameters, this is more cumbersome but would allows us to always use `params[key]` and make sure it's always the same default value.


Tinype ko ko po?


@richartkeil Ich hab das Templating ziemlich krass aufgeräumt und auch in verschiedene Unterdateien verteilt. Ich hab's jetzt, auch wenn das was größeres war, direkt zu `master` gepusht, damit wir vernünftig weiter arbeiten können. Es sollte jetzt nur noch minimale Wiederholung im Code sein. Laut Git hab ich 33 Codezeilen eingespart – find ich jut!


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (762 of 762 tests passed)

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


If this somehow passes it will be with a negative score anyway which isn't worth it.  We have many better tests qued up that need fishtest resources.  I'm therefore giving up on this and stopping the test.


Cool. Why not make this even more DRY by parametrizing the entire test on just "contentType: String" and then do everything within this method (let's rename it to checkJsonContentType(contentType: String)).

The method creates a lifter, overrides the contentType of the response with the provided arg (regardless), lifts the modified response and then what validateJsonResult already does.


Thanks, appended 74a8f8b8258d24b03f9b5a418eb48bd8bd434412.

On Sat, Sep 5, 2015 at 3:16 AM, Hubert Kario <notifications@github.com>
wrote:

> this message should go to error output too (yes, old code was incorrect)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/floatingatoll/cipherscan/commit/0728751208b9eaddd1734fe44046ee7c48705db3#commitcomment-13081787>
> .
>


Deve apenas serem editados os arquivos dentro da pasta ./src, estes outros arquivos são gerados pelo gulp.


I didn't know this, thanks :+1: Now I'll also use it (:


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Your welcome! Glad to help! Really interested in your project! :smile:


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Oh, ok, just to save more memory :smile:


Thanks


Very nice :)
You can send me emails at ceytix@mailoo.org
If you want, I can send you what I'm going to do know for the project


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Repaired stderr issue.

80 characters request declined; cipherscan does not conform to 80 lines and
functionality is unaffected.

I advise a separate Github issue containing whitespace-only reformats to
avoid complications in addressing the existing conformance issues:

```
$ git blame -- cipherscan | cut -d\( -f2- | cut -c1-18,45-50,131- | grep -v
') $' | cut -c1-18 | sort | uniq -c
  27 Hubert Kario
  17 Julien Vehent
```

On Sat, Sep 5, 2015 at 3:24 AM, Hubert Kario <notifications@github.com>
wrote:

> missing redirect to stderr
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/floatingatoll/cipherscan/commit/6b907943b56df02ab9acaaa02c29aea8a8ab2f0c#commitcomment-13081819>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.576](http://teamcity.monogame.net/viewLog.html?buildId=10081&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 538, ignored: 6 Build time: 00:21:47
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.SpriteBatchTest.Draw_with_SpriteSortMode(Deferred): <no details avaliable>

```


It would be awesome!! :smile:


hi Adrian,

I tested your code it seems the logout window doesn't close by itself.


No, as for now it just shows up in the repo. I'm sure we can add copy about
it to the "About" page.

On Sat, Sep 5, 2015 at 7:14 AM, campgurus <notifications@github.com> wrote:

> Is this going to show on the site anywhere?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BOWiki/BOW/commit/84fbed9dfc0f3881307fbd1db128ed6e125a0d78#commitcomment-13081966>
> .
>

--

Mark Nyonmark@grandkru.com

blog.grandkru.com

github.com/trystant


das ist korrekt, das muss in der Praxis parallel laufen.
Der Normalfall sollte sein, dass 2 Leute an einem Verletzten sind. Einer gibt CPR, während der andere die Wunden stopft.
wie gesagt - da ist es kurz und knapp beschrieben...
Das Diagramm wird auch noch an ACE angepasst sein. das ist eher mal ein Platzhalter.


it is, but it is needed on the same end. So no way around.


@niklasravnsborg nice!


Damit tut die Rolle praktisch nichts mehr :-1:


Do you think these names make sense? ```get_excelfile```, ```get_csv_refdf``` and ```get_exceldf```?


Wht not renaming setup_choosen_logger function too? It's static so no problem renaming it. Ideally i'd split code and comments / strings in two separated commits but it's no big deal.


I concur.


I didn't find how can i use line and column parameters of error messages in node errors. That is why i modified message.


@mortzu @jplitza @genofire Warum brauchen wir hier genau ein `-j REJECT`?
Für den Traffic aus `br-ffhb` gibt es immerhin schon eine eigene Routing-Tabelle.


This belongs into src/AutoRecordings.h


And it would be nice to have at least a comment that scale is minutes.


I hope that you can help me again. My Sungrow system stopped uploading data to SolarInfoBank and PVOutput (no idea why but it just stopped!) so I had to create a new account at SolarInfoBank and it's working again. It looks like I've got a new Sungrow User ID and Sungrow Plant ID. Please can you let me know where to find these so that I can update the script? Thank you for your help.


Could you please leave spaces around <code>*</code> and also everywhere else where doing simple math?


Fixed


The spaces "problem" all over the place...


thanks for fix that


Maybe newEnd -= (24 * 60);


Thumbs up


Yea, I didn't cosmetics yet, but I first want to know is this implementation is ok for you and @PhobosK


but why not fix the "{"plugins", "MiNET v1.0.0"}," this line,just get all Plugins Name and show it in this line


Is is working perfect for me btw ;-)


Referred to: #56


*Bank icons.


Additionally we can make it configurable too


Depends what "perfect" means. At lease use case one from PhobosK will not work, IMO.


Great!


Do not use double quotes for String  if unneeded

PS: Double quotes are a templatable String, which will either return a GString if it is templated, or else a standard Java String.
see http://stackoverflow.com/a/6761591


Package names are written in all lower case.


Hmm, why would this fail?


This really is a draft, some changes are useless, some will change again later. Just have to make my mind on how to do it.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Coverage Too Low

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


@surdupetru Could you help with this kernel?
I build cm12.1 with shipped display/media, now the panel die after lock screen.
I'm sure this commit caused the issue, but I've no idea to fix it.


Second option sounds more reasonable.


that function seems like a lot of duplicated code, couldn't you call toCvCopyImpl from it ?


Oh! Sorry for that!
I am working in this for desktop usage yet. Where you tested? (desktop, android, iPhone)?


generally


I suppose I'll send a PR if it annoys me too much.


I'm guessing that the new core-h2-all-in-one-v4.patch is a complete patch to apply to the 2.4.16 Apache httpd tree?


These tests should be something like:
```
if(sym.get_type(cursym) == SYM_READONLY) {
   if(member_is_readonly) {
      cc_error("Duplicate modifier");
      return -1;
   }
   ...
```
So you can't do:
`import readonly import readonly myFunc();`


中文注释


What's the `N_()` about here? We should be really really careful with adding l10n to PID 1. It's fine for the client tools, but I am very sure that it should not be used in PID 1 itself...


@ksooo
this one?

Let's say we have a weekday show that starts at 10.00 and ends at 11.00, but Wednesday and Friday it is from 14.00 to 15.00. The user may decide to put all at once (for ease) and the simpliest way to do it would be to set all days of week, start time at 10.00 and end time at 15.00 in Kodi.

The pvr.hts version now will fail: at Wednesday/Friday because of the "approxTime" and at all other days because of the large minDuration.
The proposed change (using mapped from the UI "start" and "startWindow"" messages) will not fail

Well I suppose the user should set start to any time and stop to any time then, as the start times are totally different. The user still can select "max once a day".


I will add toCvCopyImplHelper to avoid code duplication.


Fixes #19


MongoDB HW 5.1 test document


Given that all strings are already allocated, I think it would be OK to simply allocate the array on the stack, and reuse the string pointers we already have. i.e.

```
const char* details[8] = { "unit", u->id, "verb", verb };
```

and then later on:

```
details[4] = "polkit.message";
details[5] = polkit_message;
...
```

you get the idea.


这个repo是干嘛用的呢？没有README？


dsfsdf


test


Hmm, given that polkit namespaces its detail keys with `polkit.`, shouldn't we follow that logic, and call our keys `systemd.unit` rather than just `unit`? What's the common approach here?


'ctrlpvim/ctrlp.vim' is an updated fork.


The quote escaping was wrong here (see issue #131)


@Glenn-1990 sure, there are always different possibilities to achieve a goal. Question is how "perfect" people feel about what is possible. I could live very well with any and any for start and stop in this case, but PhobosK might feel different...

Does your current approach have problems with use case two and three?


This will probably not take the formatted time into account, might have to change it to
    <?php echo dateformat_jquery($this->config->item('dateformat') . $this->config->item('timeformat')); ?>


귀찮아서 냅둔거 해놓으셨네 ㅎㅎ
감사합니다


Lol


I have no time to try it now, but I'm not sure if it fixes the problem - now instead of no values we get empty value only, when in reality there might be some values to choose from. What I meant was to check in javascript ready() if there are any fields with no values (that would mark fields with value==None) and reload them - in contrast to render() function ajax view knows the field's parent field and can therefore get proper options even though field's value is None.

When I have some time I'll write and push this.


(or have you forgotten to include some changes? because from commit message one might think there should be some more code here...)


The Languages

C++ is well-suited for large projects because it has an object-oriented structure. People can collaborate on one program by breaking it up into parts and having a small group or even one individual work on each part. The object-oriented structure also allows code to be reused a lot, which can cut down development time. C++ is also a fairly efficient language - although many C programmers will disagree.

C is a popular language, especially in game programming, because it doesn't have the extra packaging of the object-oriented C++. Programmers use C because it makes programs slightly faster and smaller than programs written in C++. You might wonder, however, whether it's worth giving up the reusability of C++ to get the small increase in performance with C, especially when C++ can, where necessary, be written in a C programming style.

Pascal is primarily a teaching language. Few industrial programs are written in Pascal. Pascal tends to use keywords instead of C-style braces and symbols, so it is a bit easier for beginners to understand than languages like C++. Still, not everyone thinks Pascal is just for the schools. Borland, the huge compiler software company, has been pushing Delphi as an industrial strength programming language. Delphi is an object-oriented version of Pascal, and currently, only Borland compilers use it.

Fortran is a number-crunching program, and it is still used by scientists because the language allows variables of any size up to the memory limit of the machine. Fortran is especially convenient for engineers, who have to mathematically model and compute values to high precision. Fortran, however, isn't nearly as flexible as C or C++. Programming in Fortran is rigid, with strict rules on whitespace and formatting, which sometimes makes reading Fortran programs difficult.

Java is a multi-platform language that is especially useful in networking. Of course, the most famous usage of Java is on the web, with Java applets, but Java is also used to build cross-platform programs that stand alone. Since it resembles C++ in syntax and structure, learning Java is usually quite easy for most C++ programmers. Java offers the advantages provided by object-oriented programming, such as reusability; on the other hand, it can be difficult to write highly efficient code in Java, and Swing, its primary user interface, is notoriously slow. Nevertheless, Java has increased in speed in recent years, and version 1.5 offers some new features for making programming easier.

Perl was originally a file management language for Unix, but it has become well known for its use in CGI programming. CGI (Common Gateway Interface) is a term for programs that web servers can execute to allow web pages additional capabilities. Perl is great with regular expression pattern matching, which is a method for searching text. Perl can be used for databases and other useful server functions, and it is simple to pick up the basics if you have experience in any imperative language. Web hosting services prefer Perl over C++ as a CGI language because the web hosts can inspect Perl script files, since they're just text files, while C++ is compiled, so it can't be inspected for potentially dangerous code. Perl is, however, notorious for its "write once" style of code -- it's very easy to write Perl scripts taking advantage of lots of shortcuts that you later cannot understand.

PHP is a common language for webpage design that is sometimes used as a scripting language in *nix. PHP is designed for rapid website development, and as a result contains features that make it easy to link to databases, generate HTTP headers, and so forth. As a scripting language, it contains a relatively simple set of basic components that allow the programmer to quickly get up to speed, though it does have more sophisticated object-oriented features.

LISP is functional language used mostly in computer science research. LISP is unusual in that it stores (nearly) all data in lists, which are like arrays, but without index numbers. The syntax for lists is very simple, making it easy for programmers to implement complex structures.

Scheme A well-known variant of LISP, Scheme has a slightly simpler syntax and not quite as many features. A common joke is that any large project undertaken in Scheme will result in the reimplementation of most of LISP. Nevertheless, Scheme is quite popular in academic circles and is the introductory language of MIT's computer science department (and is taught as part of Harvard's introductory sequence). Scheme's simplicity makes it a good way to get started solving problems instead of worrying about programming language syntax.


Fixed in 4abf617b9c662851889de4a3ceb5aa63040dc21e. More params to be added.


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


:+1: for this version! thanks for all the great work!


I guess removing the 1.0 version would make the entire commit pretty useless :D I am still hoping they release 2.0 spec or something one day. Unlikely, but the structure is there for it.


Made a fork by mistake


It's the `-releases` patch, but I agree, the patch names are confusing. I still haven't figured out what the others are for.


btw, there is a fixer in php-cs-fixer for enforce this rule now if you're interested :)

It's called logical_not_operators_with_successor_space. Laravel uses that fixer for example. :)


P.s. you should just use preg_quote instead


This is my comment  ！


Please use  `$prosody::daemonize`


I think I now see the issue.
Please see branch https://github.com/marschap/openldap/commits/contrib-totp2
It has updates for the issues mentioned here an in the ITS.


Same here, `$prosody::daemonize`


That can't be done without beat listeners for the audio, right?


Did you mean to leave this in?


typo in commit message: independent


@jorgenavas, I think at some point you changed the code to take an optional entry-point function instead of `main`. If so, we should use that here as well.


That will definitely cause problems with puppet as it is not sure if the Service has already been evaluated, please check a variable here, `$prosody::manage_service` or similar which is used to enable the service throughout the module.


Remove the notify here completely


Yes, I am looking forward to 2.0 too, but now it may look like the user has to pay attention which features are available on which version, do sanity version checks etc. Also, for the sake of completeness, there should be also `Audio::Context::version()`, `isVersionSupported()`, version assertion macros... All of that has no real use now and would only clutter the API.

On the other hand, I'm all for extensions, checks for these, unlike version information, are really useful (or, um, unless we can assume existence of the ubiquitous ones?).

In my opinion, OpenAL 2.0, if that ever happens, would be such an overhaul that it may need a completely new wrapper, so this would be the least of the problems ;)

Keep it simple :)


I think it should be this on line 24: pPeter m p j a c = (m || j) &&  not(p) && not(a) && not(c)
The reason is that Peter said it was Matthew or Jack (and not someone else).