An explanation of this would be good for people not used to writing SSE/manipulating floats.


i send the chat by websocket successful but still have 2 problems to solve(at least)
- chat page doesn't update
- workfolw.outcome.data can't save more than one data


Else?


that's what i get now:

```
In file included from /Users/oriol/Desktop/LP/REPOS/PG_REPO/apps/PG_WALL/src/TexturedObjectScrollView.cpp:9:
In file included from /Users/oriol/Desktop/LP/REPOS/PG_REPO/apps/PG_WALL/src/TexturedObjectScrollView.h:12:
In file included from ../../OF/libs/openFrameworks/ofMain.h:9:
In file included from ../../OF/libs/openFrameworks/utils/ofURLFileLoader.h:2:
In file included from ../../OF/libs/openFrameworks/events/ofEvents.h:5:
In file included from ../../OF/libs/openFrameworks/events/ofEventUtils.h:4:
../../OF/libs/openFrameworks/events/ofEvent.h:33:42: error: call to implicitly-deleted copy constructor of 'of::priv::Function<ofxAnimatable::AnimationEvent>'
                                return std::unique_ptr<Function>(new Function(f));
                                                                     ^        ~
../../OF/libs/openFrameworks/events/ofEvent.h:156:7: note: in instantiation of member function 'ofBaseEvent<of::priv::Function<ofxAnimatable::AnimationEvent>, std::__1::mutex>::ofBaseEvent' requested here
class ofEvent: public ofBaseEvent<of::priv::Function<T>,Mutex>{
      ^
../../OF/libs/openFrameworks/events/ofEvent.h:130:35: note: copy constructor of 'Function<ofxAnimatable::AnimationEvent>' is implicitly deleted because field 'id' has a deleted copy constructor
                std::unique_ptr<BaseFunctionId> id;
                                                ^
In file included from /Users/oriol/Desktop/LP/REPOS/PG_REPO/apps/PG_WALL/src/TexturedObjectScrollView.cpp:9:
In file included from /Users/oriol/Desktop/LP/REPOS/PG_REPO/apps/PG_WALL/src/TexturedObjectScrollView.h:12:
In file included from ../../OF/libs/openFrameworks/ofMain.h:5:
In file included from ../../OF/libs/openFrameworks/utils/ofConstants.h:376:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/string:439:
In file included from /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/algorithm:628:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/memory:2515:31: note: copy constructor is implicitly deleted because 'unique_ptr<of::priv::BaseFunctionId, std::__1::default_delete<of::priv::BaseFunctionId> >' has a user-declared move constructor
    _LIBCPP_INLINE_VISIBILITY unique_ptr(unique_ptr&& __u) _NOEXCEPT
```


I need to verify all significant things that happens) GameId is very important field, that should be verified in test as all other important fields. I can do changes in code, and if you delete this line, test will be passing, but it doesn't mean that code is correct. I would avoid deleting assertions that were added for some reason


Never tried, Ill make an issue to ensure edge cases like that work.


`key: value` zamiast `:key => value`


Something closer to original (YOU CANNOT DISABLE THIS MODE AFTER ENABLING IT) would be also nice for enable-full-dev-mode-first-warning-msg-3


coverage powinien być chyba w `.gitignore`


Why is this needed?


array(\PDO::ATTR_CURSOR => \PDO::CURSOR_FWDONLY)


:tada: Wonderful!


Same comment, I'm fine splitting this out but I might be more trouble than it's worth, considering the extent of the rewrite for typed arrays and the fact that some of that code might depend on these changes (the reason they are here is that the branch I pushed is our current production branch which has been field-tested for 6 months now, splitting them up seemed like asking for bugs).


I would use `has_never.iterator()` here to save a bit of memory while iterating


Spaces not needed here, please use {[plural(count)]} in future.


Your note has sense but I won't change name. If you really need it,do it yourself.
Sorry :)


It is not at all clear to me what this does and why.  Can you explain?


:clap:


Same answer like in Configuration->Mode post :)


Instead of failing I would suggest to warn about the issue (min > max) and invert variables.

If the range is set dynamically, depending on users interaction, or something like that, eventually a developer may forget to check that on his side. But we can easily invert on our side and keep everything working.


`User registration`


This kind of escaping needs to be handled in the ``taskw`` library, not in bugwarrior please.  See https://github.com/ralphbean/taskw


Done.


I measured it with a traditional stopwatch and it seems like something about 5-6 seconds (time between destroy() and the crusher coming fully back). I think it doesn't really matter actually, a bot can place a new object after 2-3 seconds or so I guess.


@fidelram This is fine except there is a mode ``--compare``, where it simply compares set A and set B, with no assumption that A is a subset of B. I'll pull in your changes but enable the warning only when we are not in ``--compare`` mode.


`create a new user via index page`


We don't use "Naciśnij" as trasnlation for "Tap", please check https://transvision.mozfr.org/?recherche=Tap&repo=gaia&sourcelocale=en-US&locale=pl&search_type=strings and http://bugs.aviary.pl/show_bug.cgi?id=4962


Aside: I wonder if we should declare a type alias for numbers representing addresses, to distinguish them from other numbers in type signatures.


Comparing with ``str`` won't be compatible between python2 and python3.  Better to use ``six.text_type``


`validate user name`


I don't like code that is commented out. Do we want to keep this code in source code or it's better to delete it at all?


`.meta.version` is listed as the "standard meta attribute [for] the package version" [here](http://nixos.org/nixpkgs/manual/#sec-standard-meta-attributes), which also mentions that it's used by (at least) the `updateWalker`.

By contrast, `.version` isn't even mentioned anywhere in the manual. I don't understand why the monitor would use it.

Dropping `rec` isn't the main purpose here (though I don't see how it would be needed to access `meta.version` from *inside* the expression -- just use `version`?). Not exporting unused attributes is. And it can't hurt performance.

I'll change it back if you prefer, but do think either the documentation or - IMHO - the monitor need some tweaking.


It would be easier to sort this out of the jira feature was added in a separate pull request (I want to reject some of the other fixes, but this one looks okay).  Can you re-submit it?


wywaliłbym `to success...`


I'd use `has_blank.iterator()`


Players were grouped incorrectly because they were not ordered by divisions.


te trzy scenariusze można by chyba razem wrzucić i sprawdzić wszystkie trzy pola na raz


@samchrisinger, the new schemas are working however, I am getting at `TypeError: schema.config is undefined` upon selecting them from the drop down.

<img width="303" alt="screen shot 2015-07-27 at 12 23 22 pm" src="https://cloud.githubusercontent.com/assets/2614670/8911424/783deeae-345a-11e5-887a-dcfa89f5b064.png">


Forgot a closing parentheses here!


poprzednio użyłeś `'`. Zachowaj spójność w ramach projektu.


This 'if; could be important. I found once the json file that was generated by cucumber framework but seems to be incomplete. Without this check it fails on next line.

I would keep this check


What did you changed?


preferuje `let(:user) {}`


waarom is dit een array?


`blank=False` is the default, I'd drop that parameter completely to stay consistent with the other fields implying default parameter values


`log in from ...`


Unnecessary newline.


Very good point. Thx!
It will be implemented soon.


wcięcia


There's also the matter of whether the function returns binary or armored ASCII. In many file applications, returning a typed array is much better than an armored string. I think a parameter object might be better here than adding arguments, especially with the addition of passwords.

What is the rationale for supporting 'text' in addition to UTF8?


Agreement


No point in using "other" form here, it is impossible to tap 2.37 times, leave it empty (nothing after = character).


True. I'll do it


The scrollWidth property in Edge/Chrome/FF has the following descriptor:
```
Object { configurable: true, enumerable: true, get: scrollWidth() }
```

I believe you're currently setting `configurable` and `enumerable` to `false` the default.


@opdenkamp / @Jalle19 What about a separate API function 'GetRecordingsEpgList'

This could return a std::vector (unsigned int iEpgEventId, PVR_RECORDING_PTR recordingptr) of variable size (or similar).
Hopefully this would slot relatively easily into the existing code (I haven't checked)

It would definately avoid cluttering up the EPG database with blank strings or bloating PVR_RECORDING with large mostly empty arrays of unsigned ints.

This could be called directly after 'UpdateRecordings' and therfore eliminate the risk of the EPG database having out of date information.

If you think this has merit, I'll investigate further after working on some of the other changes this PR implies


This should be an empty string instead of a default. Please see below for a rationale.


Students/Vacareanu M. Robert Ionut/Chapter 2 - Classes and Objects/MatrixOperations/src/MatrixOperations.java

great! lovely implementation of Singleton


Fixing


I don't think this is a good idea. Imagine a scenario in future where we can configure two paths - datadir and rundir. Let's take into account that local config can override global config. Let's assume that we have following configuration in system config directory:
```
data: /some/data/dir
```
and in local config directory:
```
run: /some/run/directory
```
Merging those two should result in:
```
data: /some/data/dir
run: /some/run/directory
```
But with this assignment we will get:
```
data: /var/lib/rkt
run: /some/run/directory
```

So, in short, we should override `config.Directories.DataDir` only when `subconfig.Directories.DataDir` is not empty. That is why we shouldn't be setting this path to some nonempty default value.


You could grab the existing descriptor before with `Object.getOwnPropertyDescriptor(Element.prototype, 'scrollWidth')` then defer to it's original getter when not hitting the compat path you're wanting.


Same here, we shouldn't provide default values here.


Would you suggest replacing the references in core with a string literal, or actually removing those references altogether?


Would be helpful to make each address link to each apartment's corresponding show page, like so:

```
<li><a href="/aparments/#{apartment.id}"><%= apartment.address %></a></li>
```


Just a minor thing, but could the validator be instantiated once and used in both the model and the serializer?


On Mon, 27 Jul 2015, Viktor Szakats wrote:

Hi,

> This `hb_ULen()` usage will generate an RTE, because `::aText` is an array. Or am I missing something?

It's my typo.
I'll commit fix in a while.
Thank you for the information.

best regards,
Przemek


if Mockito is much better, lets use it) we should use best frameworks in our project, but if you are taking resposibility for intorducing new framework, you should take care about removal old framework


Do we really want to remove this flag? I would keep it with semantics that it has precendence over configuration.


deal with it


npm install instruction fixed


Students/Vacareanu M. Robert Ionut/Chapter 2 - Classes and Objects/PolynomialsOperations/src/PolynomialsOperations.java

`length` should be `degree`


chyba powinno być coś w rodzaju
```devise_parameter_sanitizer.for(:sign_in) { |user| user.permit(:username, :email, ...) }```


`el.style` is an object so you can store a reference to it and make modifications to the reference.


The class should be named Polynomial, as it models only *one* polynomial!


I haven't tried out Selvala yet, but one thing to test for is:

Gatherer:
"If you activate Selvala’s ability while casting a spell, and you discover you can't produce enough mana to pay that spell’s costs, the spell is reversed. The spell returns to whatever zone you were casting it from. You may reverse other mana abilities you activated while casting the spell, but Selvala’s ability can't be reversed. Whatever mana that ability produced will be in your mana pool and each player will have drawn a card."


Please split the java classes into more files, each class should have 1 file (at least for now, this should stay as a rule)


Add a radix to the `parseInt` so `parseInt(el.clientWidth, 10)`


or if float values are allowed `parseFloat`.


`spec/fixtures`


wcięcia


I _think_ you just need to have the user's public key uploaded to Github. You won't be able to push, but you should be able to pull.


i do not want to support text, but having a variable that acts as a switch leave this possibility open and in general in various points of the application (e.g. in the packet.literal the accepted strings are 'utf8', 'binary', 'text' so that i find really straighforward to use this variables instead of a boolean to express binary or not.

what is your thinking?


How much data can a single sigop make us hash, potentially? This is doing post-hash counting, if the data we're being asked to hash puts us over the limits, we only discover afterwards.

Do you think that's a potential issue?


`when user...`


Sorry about that -- since I was testing by just recompiling with `ant acesupport` I missed it. Hopefully the GWT compiler made it really obvious this was the problem?


What a great job you did! Congrats! :+1: :+1:


@drogus Do you have a link for your travis-core parallel testing experiments?


On Mon, Jul 27, 2015 at 11:16 AM, KC Sivaramakrishnan <
notifications@github.com> wrote:

> I don't clearly see the benefit in the quiescent phase. The current design
> only eagerly performs work that would anyway have to be done. Are there any
> benefits to the quiescent phase?
>
In theory, there are. The current design eagerly performs work that may
never have to be done.

It is possible (common, actually) for an object to be reachable at the
start of a GC cycle, but become garbage during the cycle. If we delay
marking as late as possible, we avoid retaining such garbage. If we begin
marking early, we will mark some such objects and keep them alive for a
cycle longer than necessary ("floating garbage").

The two extreme strategies are to begin marking immediately, or to leave
all marking until the last moment. The first spreads the work as evenly as
possible, at the risk of doing more than strictly needed, and is thus a
latency optimisation. The second does as little work as possible, but does
it all in one go. This is stop-the-world GC, which is a throughput
optimisation (no floating garbage and write barriers can be simplified) but
seriously hurts latency.

We currently use the first strategy. I'd like to provide the second as a
runtime option, since it might be useful for compute-bound
latency-insensitive applications (SAT solvers, proof search, that sort of
thing).

Having a quiescent phase gives us options between the two extremes. On the
other hand, I'm not sure those are terribly useful, so I don't think the
effort is justified.


作成者分かるのがgitの良さなのに、わざわざファイル名まで名前にする必要性ない気が。


Thanks, Nicolas.  I'm also wondering if hardcoding the path, /opt/local/, would work if Qt was installed via a non-MacPorts way, such as Homebrew or Fink, etc.  AFAIK MacPorts is the only one that installs binaries under /opt/local/.


(y) in the meantime so i will simply rebase the branch removing the edits to the /dist and we will evaluate if splitting or not in a later stage.


If you will do it in the same way as in method 'getGameByGameIdWithJoinedUser', we will throw exception if gameUser not found, can't see the difference


DRY: don't repeat yourself :) particularly lines 108ff


section 2.5:
 All Unicode characters may be placed within the
   quotation marks except for the characters that must be escaped:
   quotation mark, reverse solidus, and the control characters (U+0000
   through U+001F).


Hey Nick,

Are you still working on your Janus Plugin? I'd love to message you about it sometime at m@mattvv.com - i'm extremely interested in your results with audiobridge and how to improve it!

Thanks,

Matt


Adding a trailing comma here will make future commits easier to understand.


Ditto re: trailing commits.


(y)


Students/Modan A. Valentin/Assigment1 Prime_Spiral/src/Prime_Spiral.java

I'd very much like you to explain this approach


I think in such case we will add new logs in different wasy, as we don;t know what is correct)


is the Stack a Stack as in the Data Structure?


WTF?


Both Fink and Brew organise the files in a completely different (and much more annoying) way than Ports, so the path is hardcoded on purpose. Adding support for at least Brew is also on my todo, but I haven't managed to bother with that quite yet.


Instead of changing from calloc to malloc and setting some fields to 0, like engine, input_terms_count and output_terms_count I would let this memory block to be allocated and zeroed with calloc.
If somebody change this struct again adding a new field and forget to initialize it here it may become a hard to debug issue.
Usually bridges are created once per a program execution, so this possible small overhead of zeroing some fields that would be set to something else a bit later don't pay it, imo.


I'm fine with a string option rather than a boolean. I just don't think 'text' is a useful thing to support, except for reading, because it doesn't actually tell you anything about the data.

   If it is a 'b' (0x62), then the Literal packet contains binary data.
   If it is a 't' (0x74), then it contains text data, and thus may need
   line ends converted to local form, or other text-mode changes.  The
   tag 'u' (0x75) means the same as 't', but also indicates that
   implementation believes that the literal data contains UTF-8 text.


If one of those values coerces to `NaN` it's gonna have a bad time. Not sure if it's likely but figured it's worth a heads up.


neat


użyj `describe` kiedy chcesz opisać metodę / endpoint / ..., `context` natomiast dla grupowania speców o tym samym kontekście.. describe powinien być unikalny w obrębie pliku.
Czyli np.
```
describe "POST /create" do
  context "when user is signed in" do
    context "for valid params" do
```


w t f ?


I tried a separate sketch with ofxAnimatable and it compiled fine; so not sure what's wrong. I can privately share the project src with you if that's going to help understanding what's going on


Sorry, took that from the RFC. Text is supported by the standard, but it's meaningless. I don't think we should support writing text-mode messages, just UTF8 and binary.


:heart: :heart: :heart:


jest masa miejsc z złymi wcięciami. Sprawdź narzędzia do analizy kodu typu rubocop


No probs! Any comment on: https://github.com/harbour/core/commit/ddea7f90cb8f6c37b7e934a1b94cfb6ad40c2ffc#commitcomment-12376406


Sounds great.


`PUT /update`


wcześniej masz `let(:user)` a tu tworzysz nowego


Hooray! Code LGTM.


On Mon, 27 Jul 2015, Viktor Szakats wrote:

Hi Viktor,

> Hi Przemek, what is the reason to switch to `hb_URight()` for such strings as `::colorSpec`?

The previous version was using Right() so I decided
to not change this behavior.
Anyhow if you think it's better then we can use hb_B*()
functions to force byte oriented decoding. For sure it will
be faster.

BTW I'm not sure hb_U*() is good prefix.
    Maybe it's time to change to hb_C*() before we begin
    to widely use these functions?

best regards,
Przemek


Hi Simone, your modifications are gold! I needed to do the same to compile my project.
I think you should make a pull request in the author repository.


użyj `before`


I don't understand why this is always using the accurate measure here?


As far as I know this is a C API, not C++, thus std::vector is a no go.

BTW: and please don't ask me why we have reference parameters in the signature of some API functions. The latter does in no way fit for plain C...


Funny that the compiler didn't complain about it.


It is part of test class so feel free to "throws Exception" everywhere


no, no worries, it's just that the copy constructor is missing now i just need to add it back.


Oh, sorry, never mind. It's because the old checks are below, and pre-fork this sigop tracker won't do anything.

Let's move some of the explanations into the code rather than the commit message so this stuff is clearer.


My initial question is why Reporter and StumblerBundle need changes for an export feature, my gut feel is that this should be a modular change and not affect collection code. I'll look at this more closely in a few hours, just skimming my review requests ATM.

BTW, this is a great feature, we have been pinged about this a number of times.


to chyba nie zadziała z `config.use_transactional_fixtures = true`


TeamCity Subsonic :: Debug [Build 1](http://109.103.226.38:8512/viewLog.html?buildId=34577&buildTypeId=Subsonic_Debug) is now running


> It's valid to not have specified

It's auto magic that should never be used anywhere.

> In fact, as it stands, won't saving the config without a cap result in the value being shown as 2147483648 on next page load? That's why the UI was separating it out into getContainerCapStr.

valueStr is usually used for Lists that has no normal data-binding. Using it for String has no sense.
UI will load exact variable that was saved and MAX_VALUE will be used only for deprecated constructor or for instances that doesn't configured limit (i never used zero value because plugin had issues with leaking containers).
On new installations it will have 100 as default. This PR doesn't change behaviour, only data storage.

> <0 is probably an invalid anyway.
invalid but has no sense for if that should do simple `>` or `<` comparison. I plan review it later and empty value configuration will be migrated to new format. Though probably better do it in the same release so new constructor will have new behaviour. I will change this separately and before release.


:+1: again :)


Updated the instructions for Mac OS:

Changed the file name from "Subtitle_downloader.py" to "subtitle-downloader.py"

Added instruction to make the script take movie/video files as input


Great, just checking I understand what's going on! I'm really excited for this feature. It'll bring it closer to making sense to use bazel on floss stuff.


コンパイルも通り、複数でのデータのやりとりが確認できました。


Can you copy your commit message explanation of how the numbers were obtained into the code?

Also: halving the sigop density post-fork, it's clearly safe due to your tx scans, but you might want to call out specifically that you did that and why halving, not quartering etc.


I don't think this is tall enough? It seems to be causing the "Now go and share it!" to get cut off slightly


If you removed sml_is_ann() because it's possible to have other engines, you may use if (!sml_is_fuzzy()) return.
The idea is to avoid extra indentation for the case it's fuzzy.


I think a testcase for testing "foo\\\\bar", "foo\\\\\\bar", ... as well as "foo.bar", "foo..bar", "foo...bar" would make sense, as well as "foo\.bar", "foo.\.bar", "foo\..bar", "foo\\.bar", "foo\.\bar", "foo.\\bar" would be good.


TeamCity Subsonic :: Debug [Build 1](http://109.103.226.38:8512/viewLog.html?buildId=34577&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new); compilation error: Common.Mocks\Common.Mocks.csproj (new); compilation error: SubLastFmTests\SubLastFmTests.csproj (new); compilation error: SubEchoNestTests\SubEchoNestTests.csproj (new) Build time: 00:01:17


How comes we "break" when exceeding the new limits, but "continue" when exceeding the old?


On mobile so I didn't read it (yet). But my apps are usually namespaced (even though it is only deployed once) and as soon as I have links to the admin and my app on one site this change will break half of the links. Setting current_app to none is an option but seems quite nasty in my eyes (as it is not that easy to do for multiple views).


Thanks, but according to man page, I'd like to call lgetxattr twice. At the first, NULL pointer and size 0 will be passed to get the size of xattr. And allocate new buffer as length of (s + 1). And then we finally can get the xattr of the path or fd.

> If size is specified as zero, these calls return the current size of the named extended attribute (and leave value unchanged).  This can be used to determine the size  of the buffer that should be supplied in a subsequent call.  (But, bear in mind that there is a possibility that the attribute value may change between the two calls, so that it is still necessary to check the return status from the second call.)


It is because the header of the lambda represents its body and it is called with method body. By the way, it seems that FunctionLambdaExpression is missed here.


TeamCity Subsonic :: Debug [Build 2](http://109.103.226.38:8512/viewLog.html?buildId=34579&buildTypeId=Subsonic_Debug) is now running


Hamcrest comes with a library of useful matchers. Here are some of the most important ones.

Core
anything - always matches, useful if you don't care what the object under test is
describedAs - decorator to adding custom failure description
is - decorator to improve readability - see "Sugar", below
Logical
allOf - matches if all matchers match, short circuits (like Java &&)
anyOf - matches if any matchers match, short circuits (like Java ||)
not - matches if the wrapped matcher doesn't match and vice versa
Object
equalTo - test object equality using Object.equals
hasToString - test Object.toString
instanceOf, isCompatibleType - test type
notNullValue, nullValue - test for null
sameInstance - test object identity
Beans
hasProperty - test JavaBeans properties
Collections
array - test an array's elements against an array of matchers
hasEntry, hasKey, hasValue - test a map contains an entry, key or value
hasItem, hasItems - test a collection contains elements
hasItemInArray - test an array contains an element
Number
closeTo - test floating point values are close to a given value
greaterThan, greaterThanOrEqualTo, lessThan, lessThanOrEqualTo - test ordering
Text
equalToIgnoringCase - test string equality ignoring case
equalToIgnoringWhiteSpace - test string equality ignoring differences in runs of whitespace
containsString, endsWith, startsWith - test string matching


TeamCity Subsonic :: Debug [Build 2](http://109.103.226.38:8512/viewLog.html?buildId=34579&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new); compilation error: Common.Mocks\Common.Mocks.csproj; compilation error: SubLastFmTests\SubLastFmTests.csproj; compilation error: SubEchoNestTests\SubEchoNestTests.csproj Build time: 00:00:16


We should document these.


The policy-related methods like `policy_scope` are provided by Pundit. This is one of the purposes it serves - to only show model data to those users who are authorized to see it.

*(To be honest I'd rather the Pundit gem wasn't used until a later stage as its useful but it gets in the way of teaching and practicing some aspects of designing code when you don't have the design pre-determined by a gem.)*


The padding-top below is what would control that I believe.  It works for me locally, but I'm not entirely surprised if it's not entirely portable.  Here's what it looks like to me:

![screen shot 2015-07-27 at 11 41 39 am](https://cloud.githubusercontent.com/assets/44390/8911838/6fafde10-3454-11e5-9b35-5c828f90132a.png)


Unused return?


This should be a `tec` instead.


Can we make this string clearer? Assume you just get the error text... how will you track it down/fix it? Also - is the + index kosher?


I guess the issue is that my url is wrapping:
![screen shot 2015-07-27 at 12 41 53 pm](https://cloud.githubusercontent.com/assets/46048/8911855/f3d6581a-345c-11e5-9224-5194d13e62e5.png)


Nice errors.


Are these the same between different versions of nokogiri?


Hmm. I created this branch by rebasing. I'll check why it setup this again as a new addition to the source code.


Updated. Thanks!


Thanks for this patch. This change should go into wincompat.h though, with the other Windows-specific things.


These two conditions in line 30 are the same:
- `wiki.user == user`
- `wiki.is_owned_by?(user)`

`wiki.users.include?(user)` is intended to test if the given `user` is a collaborating user. This is different to what `is_owned_by?` tests for.


mojor -> major


Oh, good point on the + index. As for tracking down the error, `lmAssert` prints a line number and file name afaik, so that shouldn't be a problem.


Atm


TeamCity Subsonic :: Debug [Build 3](http://109.103.226.38:8512/viewLog.html?buildId=34581&buildTypeId=Subsonic_Debug) is now running


Why this? It should always be a `tec` - although `tecINSUFFICIENT_RESERVE` is probably the wrong choice.


Should this be ZLIB_HOME instead of ZLIB (it looks like it was like this before, but just noticing now)


TeamCity Subsonic :: Debug [Build 3](http://109.103.226.38:8512/viewLog.html?buildId=34581&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new); compilation error: Common.Mocks\Common.Mocks.csproj; compilation error: SubLastFmTests\SubLastFmTests.csproj; compilation error: SubEchoNestTests\SubEchoNestTests.csproj Build time: 00:00:17


(And generally software that other non-Bazel users have to work on. If it gets to "just clone and run bazel build //..." as a setup process, I'll be *super* happy.)


we also have nD for DWT and IDWT


this should be /ui/tutorial/bucket

:)


compatibility -> compatible


This is the method called to terminate a daemon. It is not called, so here it is useless.


Yeah, wasn't sure if I should leave it in or not. I can either assert/warning or remove it?


Will do.


Looks like this `it` block has been removed from `master`. Consider bringing it back in and we can figure out how to test `#make_wikis_public` together.

In future, when you have a spec that you want to write but don't have the time to figure it out, just put a pending placeholder as a reminder using the `xit` method, like so:

```ruby
xit "should change private wikis to public wikis" do
end
```

When you run `bin/rspec` the `xit` descriptions will be printed out as "pending" specs. They serve as good reminders for specs you want to write. I often write a lot of `xit` blocks when starting a new spec file. Every time I can think of a new thing the test should cover I write it down immediately using `xit` so I don't forget.


this should be /ui/tutorial/item/CAmnDJX


This loop fails because "modules_manager" is not set in self. For me this is unbelievable…


jest spora różnica pomiędzy `save` i `save!`. http://apidock.com/rails/v2.3.8/ActiveRecord/Base/save%21


@ChipLeo works on Public test realm? In my pc, dont create me the sniff...


teams -> team
update to reflect we have received permission now


'ivy` does handle it correctly, only helm fails to preselect to correct target.


zrób analogicznie jak `set_post`


This method is called by the parent daemon (the broker) to stop its external modules. So the idea is to catch it to stop our own external modules.


nie dodawałbym pustego pliku jeżeli nie jest potrzebny


Is there any possible way the client could affect the viewUrl? I thought the server was 100% responsible for it, so do we really need to escape it?


This is true when we serialize the message to send, but not when we deserialize it: receive is our and sender is their, isn't it?

We also used `sender` and `receiver` to keep the same naming as the spec.


merge  pull request fixes#78


`- @posts`


update project and then you will find out why here we can use the same way


I suggest removing the "[WIP]" tags on all of the items, the reason for this being that once something has been Dankified, then it has completed its journey of life.


sure, will do this


Nevermind. It was only changed in the conversation. Makes sense.


All the code behind is added to forward the broks to the external modules.


nie używasz wszystkich endpointów dla komentarzy. Możesz użyć `only` lub `except` żeby pominąć nie używane akcje


Fixed in f332b7b8f9a7a5e7785538bfb38a4cfacaf95c49


The jenkins job is triggered on new commits or daily. I've triggered it manually over the weekend because the last automated job failed. We can review the jenkins job but it shouldn't have changes recently.

On your side building systemvm64template work fine?


The var ZLIB is set to be `@ZLIB_HOME@` in Makefile.in, which I used for the conditional. Actually, now that I'm looking at it again, ZLIB_HOME shouldn't be used as a make variable here or in turbine - it's a autoconf substitution variable. I'll update the PR.


but you didnt verify it, you just checking is that gameId is greater then zero... it can be from 1 to N.
i have deleted it only becasuse i have changed few lines above and wanted to find out what is going on here.


I'm not sure how the `DefaultEncoder` works? My concern is that by leaving setting the encoder/decoder optionally we may allow to construct an invalid set of properties.


On principle I want all HTML to be enforced as valid.


If min == max, range would be 0, and num_terms would be division by 0.
I don't think a variable that doesn't change shouldn't even be elected as a valid input / output variable. If it's the case, we must to enforce that on sml_variable_set_range().


I can live in peace now. Lol


update permission info here as well


jeżeli post należy do użytkownika to nie jest tu potrzebne `belongs_to :user`. Możesz skorzystać z relacji postu. Poszperaj po dokumentacji ;)


Don't worry, this was just for the version not working in cauldron. It hasn't been updated.


Im on it


Oh, you silly person, you.


I have an OS X box and can run the test suite to provide feedback.


This breaks in all modman installs now :-1:


git status


This seems like a leaky abstraction and arguably doesn't respect user choice (since the user won't see what all is being synced). I'd prefer we open the Sync prefs pane and/or the Sync setup wizard rather than flipping a bit.

@rnewman can advise on the best course of action.


Instead of using the mask table you can either:
 - On SSE4.1 use the `blendps`.
 - On SSE2 use `unpacklo_ps` and `unpackhi_ps` followed by a `shufps`.
This should avoid having to keep the mask table entirely.


It seems there is support for suite sparse 4.4 in the hg tree. I will look if I can backport it


We need to flag these with TODOs I think to keep track of what is done and what isn't.

I propose putting it in the header like so:

    h1. Pages templates TODO


Setting all variables as "ids" doesn't make much sense to me.
If it's really ok to set all of them as ids, it means no prediction accuracy would be reduce. Have you tested to run the sample simulation / sample setting all variables to ids and run not setting them?
If they have the same accuracy, drop API's to set as id, just use everything as ID. But based on _create_fuzzy_terms() difference if it's id or not, I guess there is an impact.

In this case we could change our node types to have a port for input variables and another one for input variables of type ID. It's easier than creating a custom packet carrying information if it's of type "id" or not.


aha :)

thanks, works now


And a (not very high priority) followup in #610


I approve of this change


Never instantiate stubs yourself. They are not meant to be used like that.


Wonder why there were so many changes in the TS file, but not so many in the JS.


This PR has been obsoleted at least by 573644eb97c74587cad287efa195ca4132c63528, wasn't it?


If you're changing example_doc you must to change explanations about it in our doxygen (since it's used as example and explained), and also on getting started wiki page (only after merged).


This is not a valid JSON structure.
You need to add a ':' instead of '='


Oh ffs


Are you sure don't you want -FLT_MAX, FLT_MAX?
It makes more sense to me starting with range set to full range accepted by variables than using only
positive values.


TYPO!!!


You can easily save a few instructions here by moving everything to inside the switch (or maybe use an if to share a bit of code): For JustX and JustY you don't even need to load the other condition, and on JustY you get rid of the mov. (Which isn't a big deal, but still...)


I don't know what happened here, but I didn't touch this line manually.  Do what needs to be done with it.


Thanks, it looks like that was my mistake originally but might as well fix it now.


what are you re-coding?


The namespace is still provided by the `:` notation in the `viewname`. The `current_app` is only a hint to resolve any ambiguity when that namespace is an application namespace with multiple associated instance namespaces. In the case of the admin and another app, those will have different application namespaces, and the current_app won't affect them at all. Now if you had multiple instances of the admin, and the admin wouldn't provide the `current_app` like it does now, it's behaviour would be affected, and any urls would point to the currently used admin instead of the "default" admin (the one named "admin", or the last one).


Not sure I understand. It was using Right(), which changed to hb_URight() now, but AFAIU those field have no valid non-ASCII values.

I also find current prefix a bit strange, maybe this is the reason or that I didn't yet grasp when to use that I've avoided them so far. 'C' for Character seems to be much clearer.


I disapprove :laughing: Dank Matter FTW


:bug:


It has to do with the zillions of function overload declarations moving just enough that diff became confused.


On the commit comment and if I understand you.
We used to call the JTA API as Neo4J was a TM. But we don't anymore and instead we use the way other stores do it: i.e. by calling the EM Transaction API.
Correct?

If that's true, them I'm fine as long as we have some JTA specific tests (I imagine in the integration modules).


If you can't handle Vazkii at his dankest, you can't handle her at their dankless.


@benwiley @dcs619. What is the Url ( and the ImageBinaryFileId ) used for? I know we added these fields to the model a while back for you, but if it's going to be in the UI, we just need an idea of how there going to be used ( so, at the very least, documentation can be update appropriately ).
Thanks
-David


OK, no problem then. Let's leave it as-is.


Fighting @gunnarmorling for the longest name?


TeamCity Subsonic :: Debug [Build 4](http://109.103.226.38:8512/viewLog.html?buildId=34582&buildTypeId=Subsonic_Debug) is now running


Split the offset calculation in a separate variable.


TeamCity Subsonic :: Debug [Build 4](http://109.103.226.38:8512/viewLog.html?buildId=34582&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj; compilation error: SubEchoNestTests\SubEchoNestTests.csproj Build time: 00:00:21


Aww :(


Fixed in 49ca38bce0c948c8e0c125e9a830f41e319678ca.


TeamCity Subsonic :: Debug [Build 5](http://109.103.226.38:8512/viewLog.html?buildId=34584&buildTypeId=Subsonic_Debug) is now running


I don't see any division by zero here. for non id fields, num_terms will be zero and no term will be created. For id fields num_terms will be one and a single rect will be created. Maybe taking the second approach for both is better way to go.
Well, It is weird to have min == max, but it should work fine, since the min/max value would be a valid value for the variable. I don't think we should force set_range to forbid this.


Do the cool kids still use `xor rax, rax` to zero a reg?


TeamCity Subsonic :: Debug [Build 5](http://109.103.226.38:8512/viewLog.html?buildId=34584&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj Build time: 00:00:21


It's the kind of feature that really could add a lot of relevance to sml.
Making machine learning general purpose and easy to use isn't easy task.
Let's try to give this algorithm an extra round of review / work... we can assign Iscaro or myself to review it, maybe an extra pair of eyes on it may give us some ideas to improve it.

Is it blocking any other feature to be implemented?


This deletion is not related to the Shift-Click behavior,
but I found the line to be useless.
I'm planning to remove more superfluous comments soon,
as my coding skills grow overtime :-)
I have a tendency to "over comment" things,
and I'd like my code to become progressively cleaner and cleaner.


Yes, that somehow made optics launch in 1.8. Though I'm getting a weird crash that tells me the assets from optics are not being loaded


I think you mean 3dank(Sqrt16)me. Get with the times.


Suho: Can a worker node also become a manager and a coordinator? Why I'm asking this question is that the distributed lock can be acquired by any of the nodes in the cluster including workers and managers.
Thanks


Actually, going back to electronically translate this line, I'm not sure how this translation came to be.

I THINK it actually should be: "Agailla Flurry's Shield," based on the context of the quest and the use of the term Flurry.


Top notch. Thanks, @stapelberg !


TeamCity Subsonic :: Debug [Build 6](http://109.103.226.38:8512/viewLog.html?buildId=34586&buildTypeId=Subsonic_Debug) is now running


ooops, thanks for fixing that.


With the current system, differences in the low-level input can cause large differences in the high-level results (e.g. different detected genres). Because of this, it's important that we match highlevel with its exact low-level output. Linking through the ids is a good way to do it.


I don't like this.


@tandrus Thanks for the pull request. I will pull it in, but should give it about 1 week as this causes a breaking change if not on latest node version.


TeamCity Subsonic :: Debug [Build 6](http://109.103.226.38:8512/viewLog.html?buildId=34586&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new) Build time: 00:00:10


Checking new comment


Also, are non eax->rax moves in x64 implicitly zero-extending? In x86 moving a value to ax/al/ah didn't clear the upper half of eax.


This patch addresses the use case where the user doesn't want to sync anything but does want to use Send-Tab-to-Device. Flipping the services.sync.enabled to true won't cause anything to be synced, it will only allow Send-Tab-to-Device to work. The disabling of services.sync.enabled when no sync services are opted-in is a recent change coming from Reading List work.

Also, sending the user to the Sync prefs page isn't enough. It's the services.sync.enabled pref that needs to be "fixed" and that pref isn't exposed (directly) in that panel. Will running the Sync setup wizard allow us to reach a state where nothing is getting synced but services.sync.enabled=true?

How about I additionally check if services.sync.enabled=false and some of the sync services are also (somehow) opted-in? If that's the case, I can send the user to the Sync prefs page and ask them to manually flip the pref?


I agree that it's not really a DB-related function. Perhaps it's more related to views.


You shouldn't be getting this information from the visitor.


Hello, I don't understand why you revert?
It makes the compilation not working on linux...


TeamCity Subsonic :: Debug [Build 7](http://109.103.226.38:8512/viewLog.html?buildId=34588&buildTypeId=Subsonic_Debug) is now running


Why is this back?


Need to check why return black screen when use 'boot', 'load' or 'menu''


TeamCity Subsonic :: Debug [Build 7](http://109.103.226.38:8512/viewLog.html?buildId=34588&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new) Build time: 00:00:10


Max for a single sigop is a SIGHASH_ALL single input transactions with lots of outputs. You'll hash size of serialized transaction minus size of the scriptSig-- if transactions are allowed to be as large as the block size, block size minus 70 or so bytes.

Worst case for a transaction with multiple sigops is... uhh... I dunno, I'd have to  think pretty hard about the limits on scriptSig and scriptPubKey size. Worst case actually found in the chain is a almost-1-MB, 5,500-inputs, single-output transaction; each input required 215,000 bytes to be hashed (that'd be the size of the transaction with the scriptSig's cleared) for a total of 2.1gigabytes hashed to validate that one transaction.

RE: post-counting:  CreateBlock does the right thing, calling ConnectInputs and checking to see if the transaction is valid with respect to the current block before adding it, so no, no potential issue. I should write a regression test, though, to be absolutely sure there's no bug (it is time consuming to write regression tests that run up against such high limits, you need either very large or non-standard transactions to get close to them).


Should not be in the visitor class.


Ok, I understand this is a temporary hack to test your patches but this not in shape to be merged at all... I'm cooking a patch for this.


just get(...) or if you can't protect/private, then _get(...)


Ist gefixt. Alle sourcen haben jetzt ne BOM. Damit sollte sowas nicht wieder passieren.


This is racy. If the xattr changes between the two lgetxattr() calls this will fail. I am pretty sure this should be an allocation loop: allocate a buffer of some sane size, try to read the xattr. If it works, fine, return it. If it doesn't, double the buffer, try again. See how realink_malloc() already implements this.


It is necessary to change in the other test scripts too: oceanviz-test.rb and thrusters-test.rb


@JerryCai I doubt anybody has changed anything. It looks like your plugin is still in the update centre.

If you believe your code is better than an existing plugin, the correct thing to to would be contribute to that plugin.  Having multiple plugins that do the same thing just wastes development and maintenance effort and adds confusion for users.  Users shouldn't have to seek out multiple plugins and compare them all to see which is the best.


@vargaz when I run this with 4.0.9.13 I see the following and the test hangs

exception inside UnhandledException handler: Thread was being aborted


Ah D'oh! The problem with the earlier commit is that the size was incremented anyway incremented anyway.


And I think you can directly compare the reg to mem without moving it first.


either way, just a matter of concept here, simple is a target so shouldn't be initialized like this, I'm wondering if it's even working at all, aren't you getting a rule redefinition warning?


I'll add a better version


I'd change this to be more customer focused.

You should use transaction to demarcate your work in Neo4J (is that said already in the doc?)

In the case of JTA, Hibernate OGM attaches the Neo4J internal transaction to the JTA transaction lifecycle.
That way when the JTA transaction is committed or rollbacked (for example by an EJB CMT or explicitly), the Neo4J transaction is also committed or rollbacked.
This makes for a very nice integration in a Java EE container.


Rather than duplicating the `text-decoration` rule inline, better to add a CSS rule for `a.fangorn-toolbar-icon`.


```java
Shenanigans Shenanigans = new Shenanigans(Player victim);
if(victim.getName("Pahimar"){
    while(true){
        Vaskii.performShenanigans();
    }
}
```


I'd start the caution by something like

Be careful, this is not a true JTA/XA integration but more a lifecycle alignment.
In particular, changes on more than one datasource won't be executed as a single atomic transaction.


Skąd klasa form-result2? Czy to jakaś planowana na przyszłość? 2 nic nie mówi szczególnego. Powyższa form-data była chyba bootstrapowa. Proszę o zdanie wyjaśnienia :)


Ok, but I wouldn't call this target `simple`. Maybe `samples-generator` would be a better name for this target.


This test failed for me locally on master as well, which is why it's commented out.


Ok. thanks.


all right.


erm... well, I notice that you appear to be doing source and destination differently than the starter kit, for starters...


It works and i could not see any warning.


Instead of doing the attr >= SMACK_ATTR_ACCESS I'd just do "attr >= 0" here, since that way we don't encode that SMACK_ATTR_ACCESS is actually the first name in the enum. This works nicely since in C enums are just integers, hence we can either write them as integers or via their names, it's equivalent. Or in other words:

```c
assert(attr >= 0 && attr < _SMACK_ATTR_MAX);
```


TeamCity Subsonic :: Debug [Build 8](http://109.103.226.38:8512/viewLog.html?buildId=34590&buildTypeId=Subsonic_Debug) is now running


After run this, it will be generated some .osgb files.
It would be good, to put a line in .gitignore to ignore the files with .osgb extension.


The idea was to do the custom packet types later. I should have added a TODO here. But this port strategy is really easier, so I'll implement that.


TeamCity Subsonic :: Debug [Build 8](http://109.103.226.38:8512/viewLog.html?buildId=34590&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:47


Très bien !!! ;)


aren't you breaking current sol-fbp-generator users with this?


The general system I want to get into here is that all calculations happen in models, or a view. A template should have only data display and basic flow control.


@Test(expected=Exception?.class)


Oh wow, this is old. If nothing breaks, fine by me, but I'm a tad far from flow code lately.


No, none o them.


Формально парсер сейчас не только парсит, но и что-то пишет. Думаю, стоит либо переименовать класс и метод, либо пусть метод возвращает строчку, полученную в `String.format()`, а пишет в нужный файл уже тот, кто его позвал.


Хранить `inputStream` нет смысла, т.к. его старое значение нигде не используется. Лучше создавать заново локальные переменные.


This looks wrong. The read stub call should return -ENOSUP or so, because returning 0 would indicate that *label is actually initialized.

For the writer call it's fine to turn them into NOPs returning 0 if HAVE_SMACK is off, but the functions that return something really need to return some useful data or an error, but cannot return success and not initialize their return params...


Предлагаю перенести открытие эдитора из `ModelNameView` вот сюда.


Если перенести открытие эдитора внутрь метода `initialize()`, то этот метод тоже можно будет убрать.


tmpInt = *(static_cast<int32_t*>(&bytes[byte_offset]));
I know, so ugly.


Hum... Why ?


same here.


+1. Just remember change the `simple` to `samples-generator`


Looks good otherwise. Love how this patch improves things!


Еще более красиво будет сделать классы `ProjectConfigurator` и `ConfigParser` нестатическими и создавать их инстансы отдельно для каждого пользователя, т.к. никаких данных между использованиями им хранить не нужно.


Every time Pahimar logs into MC he should get a message with some Dank meme, that's what you're saying melo right :P


ok, now I see, you're keeping backward compatibility in the block [588..594]


If mac_smack_read() returns EOPNOTSUP on non-SMACK builds, then this needs to be caught here of course (see other comment)


I revert the changes i latest push, I put the method in refactoring class


Or maybe all he needs is a fancy pointy hat. At all times. Whenever he's playing with botania on.
*hint hint*


TeamCity Subsonic :: Debug [Build 9](http://109.103.226.38:8512/viewLog.html?buildId=34592&buildTypeId=Subsonic_Debug) is now running


Creating README file for documenting process for running a python virtual environment.


TeamCity Subsonic :: Debug [Build 9](http://109.103.226.38:8512/viewLog.html?buildId=34592&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: SubEchoNestTests\SubEchoNestTests.csproj (new) Build time: 00:00:16


Ah, I just saw that `current_app` does not raise an error, ie `reverse('admin:index', current_app='admion')` resolves to the default admin instance, so that mostly eliminates my fear.


but do you really need to allocate a new string for this? can't you accomplish the same without [re]allocating the function string?


If it works, a very welcomed improvement, thx!


could this use
    idx < _int_vector.size()
(as in, no this, and start with an underscore)


Ok ca roule


Can this just be a thing?


コマンドの挙動は問題なかったです。
コマンドの説明も伝わるのでいいと思います。


TeamCity Subsonic :: Debug [Build 10](http://109.103.226.38:8512/viewLog.html?buildId=34593&buildTypeId=Subsonic_Debug) is now running


I don't follow, as I wrote this sentence in response to that review. Why is it not surprising to find negative curvature in a space that mixes quickly, particularly now that we know that fast mixing is proven rather than just likely? I don't see anything in that sentence or any of the reviews that suggest that any particular pairs of trees are difficult to move between in the absence of data, and we were certainly surprised. The one problem I see with this sentence is that we refer to "bottlenecks" but we already changed that to "paths that are difficult to traverse stochastically".

I suppose we could change this to "We were surprised by this result, as the Markov chain ...".


> go


On Mon, Jul 27, 2015 at 09:40:34AM -0700, WaLyong Cho wrote:
> Thanks, but according to man page, I'd like to call lgetxattr twice. At the first, NULL pointer and size 0 will be passed to get the size of xattr. And allocate new buffer as length of (s + 1). And then we finally can get the xattr of the path or fd.

That is a bad interface unfortunately. There's a race, and the attribute might
be modified in between the xattr accesses.

Zbyszek


I guess I could just inline the whole thing to be:

```
rec.int_vector.push_back(ntohl(*(static_cast<int32_t*>(&bytes[byte_offset]))));
```


TeamCity Subsonic :: Debug [Build 10](http://109.103.226.38:8512/viewLog.html?buildId=34593&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


Vaskii pls


Thanks for finding this.  I fixed it in the source and forgot to compile.  walkAST calls visitor methods on this.loopProtect which modify the AST.


That does not work as it should! The interesting part is when you press `M-p` to get the previous element of the history which is the same as :preselect, that is (car helm-make-target-history), "all" gets selected, but isn't initially by preselect.


also I think you're missing a migration for generate_node_specs()...


I've reverted this, as this is not neccessary.

The default favicon location [/favicon.ico](https://startupweekendrheinmain.de/favicon.ico) is redirected to <https://startupweekendrheinmain.de/bundles/foundershubstartupweekend/images/favicon.ico> by the webserver configuration,


I made some changes (I hope they are what you expected).


I agree with you that this would be a really nice feature and I think that we should work on that.
I removed this code because the idea implemented is not a good solution the the problem. It is not an implementation issue, but the algorithm proposed is not good.
I thought that this implementation would block the new implementation of automatically created terms, but I just found out that this is not true. I'll remove this patch from the series for next version of this PR.


Is this safe against shell injection?


super nit: this code is generally nestling * and & with the varname, not the the type. (although I prefer type nestle myself)


Can't this be removed entirely?


@johanhaleby You probably don't get much out of the google analytics badge on the readme within github. Github uses a proxy for the images and partially caches the content to limit the exposure, so you're going to see lower numbers than people who actually see the image. If you're a project member you should be able to use [github traffic](https://github.com/jayway/rest-assured/graphs/traffic) for good analytics.

All this being said, you will benefit from including this image if the readme appears on any other page. You'll see hits from non-github forks, hostings etc that keep the image in-tact.


ok. It makes sense to me


See below. :)


Yuss :P When do we go to factorization to make Dank iron ore?


Oops, sorry :p


by-value


It's better not to skip this. People are really frustrated when an example doesn't work. I remember it was some instruction to have hidden lines of code, you can use it to import everything.


Like we talked this -f option doesn't seem right to me, it looks more like a design conflict here with some missing parts  included, @cmarcelo may comment on this.


原图是 28x34。 @2x png ，这个大小更好。


The swizzle is known at compile time, so you should just use `shufps` here and skip the table.


TeamCity xBIM :: XbimExchange :: Build [Build 373](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2140&buildTypeId=XBIM_XbimExchange_Build) is now running


update_sqlite_library (mbadams5) - 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-100.0%25-green.svg)


```run-tests.bat``` is just crashing on me do I need to run it from a specific folder?

```
'getbranch.bat' is not recognized as an internal or external command,
operable program or batch file.
Building Debug Testsuite
Compiling testsuite...
Microsoft (R) Build Engine version 12.0.31101.0
[Microsoft .NET Framework, version 4.0.30319.34209]
Copyright (C) Microsoft Corporation. All rights reserved.

MSBUILD : error MSB1009: Project file does not exist.
Switch: ..\VS2010Express\XBMC for Windows.sln
The system cannot find the path specified.
------------------------------------------------------------
---------------
   ERROR ERROR ERROR ERROR ERROR ERROR ERROR ERROR ERROR
---------------
ERROR: "-test.EXE failed to build\Projekte\xbmc\project\..\vs2010express\XBMC\Debug Testsuite\objs\XBMC.log"
------------------------------------------------------------
```


I guessed "Compile", but yeah, it doesn't need to be abbreviate like that I don't think...


For helm, `helm-make-target-history` is actually the input history, so you type "al" then select "all", helm will try to preselect "al", of course it will fail and just use the first one. In my opinion, we are just using helm in a wrong way.


update_sqlite_library (mbadams5) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Then whatever's requiring it to be upper casing should be fixed.


See https://github.com/JuliaLang/julialang.github.com/pull/255


It doesn't seem necessary to pass around min_hit_support since it is part of the class definition.


Okay, thanks. Then what should we return? The size of return or 0?


yup, although, assigning to a tmp is very handy if you think you might be debugging in here


TeamCity xBIM :: XbimExchange :: Build [Build 373](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2140&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:59


这里，原来的实现是有问题的，bubbleFrame 是整个大的区域：

![image](https://cloud.githubusercontent.com/assets/5022872/8912846/e8dccd20-34c7-11e5-8f46-edbd2a703a54.png)

textFrame：
![image](https://cloud.githubusercontent.com/assets/5022872/8912850/ee79668a-34c7-11e5-85fe-f74691986041.png)

所以高度设置成`CGRectGetMidY(textFrame) - CGRectGetHeight(animationVoiceImageViewFrame) / 2 `


谢谢关注


go run gohtml.go
go run: cannot run non-main package
?


You mean that my paths start with `../`? That's because if they don't, Gulp panics, saying that it can't find the files at all. I spent a while struggling with that issue; I am not really very familiar with Gulp at all.


I'm not qualified enough to answer that definitively. As the comment notes, the `start` command simply doesn't work with `rawSystem` (at least, not when I tested it).

If that's a dealbreaker, I've seen [an alternative approach](https://github.com/Cadynum/Apelsin/blob/5fcf62102d8687e49eea91d8885eb5bc3fb5d594/src/Constants.hs#L46-51) where some used the `wscript` command and a [short JavaScript file](https://github.com/Cadynum/Apelsin/blob/5fcf62102d8687e49eea91d8885eb5bc3fb5d594/share/open.js) to handle opening pages in a browser. I'd be willing to try that if shell injection is a concern.


Is handling this really necessary? I know it doesn't hurt, but it's not like you do it for any other instruction.


A link back to the Apartment show page would make the UX for your Tenant index page a little better!


What circumstances are you running check_po under?

Python2?  Python3?  What o.s.?  What terminal?  What fonts?  Etc.

I suspect you don't have some font installed (if you are running it
under Ubuntu, etc.)  My o.s. doesn't do it like that (all of them are
there, I guess).

I have no crashes running check_po on either el.po or vi.po,
using the check_po I downloaded yesterday, using either Py2
or Py3.  I ran them inside an emacs shell, so I didn't have to
worry about making sure I was running my xterm with a font
which had characters for those languages.  (Emacs loads in
all the fonts, I guess.  At any rate that's how I ran check_po
on the Chinese po files a few months ago, when I couldn't
find a font to manually start my xterm with.)

My guess is that check_po doesn't need any changes.
Or if it does, it needs some test to make sure the user
has the correct font installed for the script they want
to test with check_po.


Erm... no. More like, you have dest in your Vulcanize properties, your source is actually in ../src/ whereas the example first copies the source to-be-vulcanized into dest and then runs vulcanize with that as the source, and also you don't end with a pipe to a directory, which the sample does


It looks as though the string manipulation in OnUnhandledExceptionHandler is resulting in a call to  System.Globalization.CultureInfo:get_current_locale_name() which is causing the exception

To test this, if I add the following to the top of main then we no longer see the exception and hang

var c = CultureInfo.InstalledUICulture;

Not a solution I imagine but it might give some insight into why we're seeing exceptions within the unhandled exception handler...


It is unclear to me whether this belongs in the sequence here.


Pad this one with zeroes so it doesn't look weird.


The return value ($f) should be checked here, or alternatively, use Genome::Sys->open_file_for_reading which will confess if the file cannot be opened.


コミットのコメントをし忘れてたが、visual studio を開いたときのソリューションエクスプローラーでchapterなど、少し階層別に表示されるようにした。


anything >= 0 for success is fine. But yeah, returning the size would be particularly nifty, as we'd return useful data then, instead of just "0"... I figure we could change readlink_malloc() in a similar way too...


Noooo


Doesn't look like you use `SCRATCH` after this... Bug?


like, you want to be writing something along the lines of:

```
return gulp.src('../build/index.html')
    .pipe($.vulcanize({
        stripComments: true,
        inlineCss: true,
        inlineScripts: true
    }))
    .pipe(gulp.dest('../build'));
```


See if that fixes it?


Could the _count_cols function be renamed to 'get_id_idx' and include this functionality? It is only used for this purpose.


Hmm, so the function you define is called `getxattr_malloc()`, but it wraps `lgetxattr()`. That's misleading. Either the function should be renamed to `lgetxattr_malloc()`, or it should get an extra boolean parameter `bool follow_symlinks` that allows choosing whether `getxattr()` or `lgetxattr()` is invoked internally.


Good catches! Sad that I cannot discriminate between Heard Library and Central Library by this point in my career at Vandy.


I'm not sure that changing this naming scheme is helpful - historically anything called <name>Digis always refers to the unpacker versions, while sim<name>Digis refers to the simulated version. All the other subsystems appear to follow this naming scheme, so I vote against changing it just for the RCT subsystem.


Can you return a NULL here so that user has some way to know something went wrong?


To make this obvious I'd stick to the libc nomenclature here, and call this `fgetxattr_malloc()` rather than `getxattr_malloc_fd()`.


It should just use the jar, but it might not. I'm on holiday...


You can do this with shifts to clear the bits instead, avoiding the need to load and apply the masks.


Because of this issue: https://github.com/rrthomas/lrexlib/issues/20


This is xbuild that require it. A fix could be to downcase the file name (BakaBT.png --> bakabt.png).


This should be *omitted* rather than *ommited*. :wink:


How about returning by default `"actor#ID"` instead of just `"actor"`? This would make it possible to distinguish different instances. If users don't want the name, they can still remove it via post-processing but it's not possible to do that the other way around.


You are right. I forgot this line and it's bad. I'll fix it


More like a "TOCHECK". If anything uses crappy approximations, I'd expect it to be a embedded GPU.


Added the test for FunctionLambdaExpression case. Thanks!


Ignore previous comment!


^ this. Unless the other code crashes with such configurations, there's no need to optimize for pathological inputs.


Wait why are we removing agenda?


But why? D:


Where is it called from?


For "view sample apartment", pick an Apartment id that you know will exist (i.e., 1,2,3 - one of your seeded apartments). Using 20 as an id returns an error since my Apartments table hasn't generated that id yet.


A few edits and one comment:

Build jobs compile your project in preparation for deployment. They generate artifacts that can be sent to a build archive directory, although by default, the artifacts are placed in the project's root directory.

Jobs that take input from build jobs must reference build artifacts in the same structure that they were created in. <!-- LWH: If by "reference," you mean "refer to," then replace "reference" with "refer to." --> For example, if a build job archives build artifacts to an `output` directory, a deploy script would refer to the `output` directory rather than the project root directory to deploy the compiled project.

---

Deploy jobs can deploy new apps or update existing apps. Even if you first deployed an app by using another method, such as the run bar in the Web IDE or the Cloud Foundry command line interface. To update an app, in the deploy job, use that app's name.

---

**Tip:** To deploy a previous build of an app, from the **Stage History** menu, select the app. Click **SEND TO**, and select a target.


yupp, wasn't sure about this either, hence i just declared it "food for thought"... ;-)

Might be something to document somehere though, for example next to the variable declaration, clarifying this.


Indeed, but the trouble is, I have found numerous uses of `rctDigis` which refer to what might better be called `simRctDigis`, and so I can either avoid conflict with existing code or overhaul all the naming.
I might just tackle the latter, we'll see.


Unbelievable how much SFINAE boilerplate one has to write these days to get a simple introspective trait that checks for member existence.


Oh, one more thing: I think we should use CLOCK_BOOTTIME or these timestamps, after all the timeouts should not be paused during system suspends, since they are about traffic seen outside of the system...


is it right ? Shouldn't be sizeof(fieldX) - 1 ?


Use the `CMP_LT` constant.


This doesn't seem right. The parameter is called "declaration". LambdaHeader is not a declaration. If you handle lambda header here you'd need to handle all query lambdas as well.


Your new route (and subsequent create route) does work, but you didn't include a link to it anywhere in your app!


sorie


LOL! I'm getting ready to announce it to the public. But I'm waiting for the brew package to get merged.

Until then the readme doesn't reflect reality. I don't want people to get upset when they follow the readme and it doesn't work as expected.


I'd be fine with the change if you renamed the method to GetLambdaOrMethodModifiers


please call this `internals.rust-lang.org` and end the bullet with a period, to match the existing style.


update_sqlite_library (mbadams5) - x86_64-MacOS-10.9-clang: OK (1757 of 1757 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)


seems to be causing NPEs with `compiler:spec` https://travis-ci.org/jruby/jruby/jobs/72773792#L3269-L3276


Trying this now, thanks @rcrowder.


is a completion block necessary for this function. It just seems like it is a little more complex than it needs to be. Is there any reason we can't just return a TGRouter or nil from this function?


test_fit_tail fails here after these commits....


:ok:


NAMEコマンドの処理、問題なく動きました。
他のユーザに名前の変更を知らせる点では、後々POSTコマンドを作成したら、使用すればいいと思います。


Since the last number found is 49=7^2, to get to the next number on the top right diagonal you need to add the lenght+1=8. and we check if it is prime.Then to get to the top left we need to add 8 and again check if it is prime.We also add 8 to get to the bottom left number and again to get to the bottom right number which should be a odd square.
So for the first test 49+8=57(should be on right top) j=8;
57+8=65 (left top)
65+8=73(left bottom)
73+8=81(right bottom)
then J becomes 10
81+10=91
91+10=101
101+10=111
111+10=121
then j becomes 12 and so on (J=J+2)
and we check each number if it is prime by using the vector a[] which is 1 if the number is prime
and 0 if it isn't. and on the total number we increment with 4.
nr=number of prime numbers found
total=total number of both diagonals
j=length+1 which you need to add to get on the next point on the spiral on the diagonal.
Also j=j+2 because when you do a complete spiral you the number of rows and collumns is increased by 2.


on line 68 when you do Wire.read() does it automatically take away the header and other formatting stuff?


@elect86 It's been years since I worked on this code. Sven's the authority on it.


Spelling (omitted)


Troubled *pass*? Shouldn't it be *past*?


Machnij to ładniejsze


VS2015 has new SAL checks VS2013 didn't.


This is right: `sizeof("foo") -1` is 3. This changes the character after "o" of the "fooX" string.


The first thing the Quick Start section - several lines below this - does is link to the book. This seems redundant to me.

I might also put this sentence as its own paragraph *after* the paragraph saying 'This repo contains'; that way the structure of the opening is 'rust is ...' -> 'this page is' -> 'but you probably want this other page'. Unfortunately that puts the new book link down even closer to the existing book link.


Currently does save new tenant information. Some steps to fixing it are (1) replacing `index` with `tenants` in your post rout and (2) creating a Tenant using `params[:tenant]`, not `params[:apartment]`.


Still not accurate.
* I already told you what the endgame values should be in your array
* If a value is 0, it will not show in the tuning


while(nr*10>total) // check if the number of prime number is less than 10%
+		{
+			for(u=0;u<=3;u++) //we check all the new 4 numbers from the diagonals
+			{
+				n+=j;
+				nr+=a[n];
+			}
+			total+=4;//we have 4 new number on the diagonal
+			j+=2;//Since the number of rows and collums is increased by 2 we also increase it
+                     by 2.
+			i++;
+			//System.out.println(nr+" "+total+" :"+n+" +"+(n+j)+" +"+(n+3*j));
+		}


you are un-doing my fix!  Do you think this is not needed anymore with the new setup?

https://github.com/JoelCDL/public_interface/commit/9d8145de4cef74cca18e7672aa7c0a4e8f010833


In theory I agree with you, but especially when we get Hive support (#93), this will basically be a requirement.  Also, `thrift_sasl` actually does not itself depend on `python-sasl`...it's very lightweight.  It used to simply be a part of the repo, but we've factored it out in an attempt to harmonize that code across a few other places in CDH.


What's `sfMethod` here? Can we have an enum or a named thing? Also, I'd personally avoid the `switch` in favor of:

    if (ctx.tx[sfMethod] != SYMBOLIC_CONSTANT)
        return temMALFORMED;


@lockdrew Wire.read() just gets the next bite. There isnt any header info being passed.


Good idea. We could even claim a page to do if, so inclined:

  h1. Pages templates [todo:dw]


TeamCity xBIM :: XbimExchange :: Build [Build 374](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2141&buildTypeId=XBIM_XbimExchange_Build) is now running


Ok


ok


The ENC analyzer distinguishes between "body" and "declaration". Ideally we would have that distinction in type system, e.g. we could wrap syntax node into a struct. For now we just use names.

We can rename "declaration" parameters of IsAsyncMethodOrLambda and GetModifiers to "declarationOrBody". Or we need to handle these separately in the caller of IsAsyncMethodOrLambda


TeamCity xBIM :: XbimExchange :: Build [Build 374](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2141&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:25


I'm fine with the number *1*, until we add a second method then it can get a name. `switch` makes sense here


Again why a `1` here? I thought that it maybe indicated the specific type of suspended payment operation (e.g. "create" vs. "finish" but that doesn't seem to be the case.


You are right, I haven't noticed it before.
But it's still strange why the second column with the meta data is not shown.


Might be a good idea to rename this to `swiftDefaultValue`, and similarly for any others. Then all the Swift-related properties share a prefix. I had intended to do the same with Java properties, where I had to add things like this.


I think you may need to mark datasets imported from a history archive and exclude them right? https://github.com/galaxyproject/galaxy/blob/dev/lib/galaxy/tools/imp_exp/__init__.py#L246


add_fixtures_to_VS_solution (mbadams5) - 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-100.0%25-green.svg)


I like it, will do


LGTM otherwise.


@dalehotchoco Download what?


add_fixtures_to_VS_solution (mbadams5) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Done and done. Sorry for the delay; just had some major stuff come up. Merging now


Fix .js-simple-tweet .stats


Yep.  We talked about that at some point I think, thanks for reminding me.

On Mon, Jul 27, 2015 at 1:53 PM John Chilton <notifications@github.com>
wrote:

> I think you may need to mark datasets imported from a history archive and
> exclude them right?
> https://github.com/galaxyproject/galaxy/blob/dev/lib/galaxy/tools/imp_exp/__init__.py#L246
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dannon/galaxy/commit/4afc756c6122a71248924f1f6a4bd093a7fc95b6#commitcomment-12378655>
> .
>


How about `rawSystem "cmd" ["/c", "start", url]`?


There is no instructions about this points:
https://github.com/jsdelivr/libgrabber#examples

PS. Может вы напишете инстукцию на Хабре, как добавлять и обновлять библиотеки?
Какие нужно соблюдать при этом соглашения?


Edit to my edit:
Deploy jobs can deploy new apps or update existing apps. Even if you first deployed an app by using another method, such as the Cloud Foundry command line interface or the run bar in the Web IDE, you can update the app by using a deploy job. To update an app, in the deploy job, use that app's name.


I'm confused...what exactly are you asking here? Is the code failing?


Hah.


@RagnarokNova I guess the new elemental bracket thing is only available on newer clients (2015)


So they've added a useless annotation-bug, [but haven't fixed actual, bad, annotations](https://connect.microsoft.com/VisualStudio/feedback/details/1114418/bitscanreverse-as-defined-in-winnt-h-line-5065-causes-analyze-to-generate-false-positives-with-proposed-solution)??

That's pretty terrible.


s/option/action/


Um, that's wrong, the source html file is not in `build/`, it's in the root directory (in this commit I had moved it to the `src/` directory to see if putting it in a subdir would fix it).

I copied and pasted in your code, changed the source path to point to something that exists, and it still outputs nothing.


This function is getting pretty seriously meaty. I know you have some state here, but maybe it's worth breaking this out?


http://doc.qt.io/qt-5/qscreen.html#devicePixelRatio-prop

> Common values are 1.0 on normal displays and 2.0 on "retina" displays. Higher values are also possible.


This specific commit is wrong, we follow FreeBSD style(9) and it says multiple lines should be indented with 4 spaces. You can also see this rule at pfSense Developer Style Guide at https://doc.pfsense.org/index.php/Developer_Style_Guide


I would call them `options_callback_name` and `root_type_function_name`


I had changed it to be lower case but obviously something squashed the change out of existance, its now lowercase as the rest of the logos and as per the project file.


Why use ```unassignItem``` + ```removeItem``` and then ```linkItem```?

https://community.bistudio.com/wiki/unlinkItem


Should this be `return except(std::forward<F>f, "");`?


Sounds like a plan!


float 1.0 is represented by the bit pattern 0x3f800000, so you can just use `pslld` and `psrld` to shift the all-ones mask set by `CMPSS` right by 25 and left by 23 to get that.


I said that the example in the Polymer kit `copies the source to-be-vulcanized into dest` and _then_ runs vulcanize with that as the source.


... and here ...


1 - There's no need of allocation in this case. Feel free to remove them.
2 - I don't like -f either.


Thanks for explanation. I will rename for now. Thanks!


... and here?


Let's do it. It seems it's the best solution for now.


Breaking change = major bump indeed. But I have not read any code for over a week and won't read any for one more week cause I'm building a ceiling now ;) so this is just a general comment

-------- Oorspronkelijk bericht --------
Van: Kai Sommerfeld <notifications@github.com>
Datum:27-07-2015  17:01  (GMT+01:00)
Aan: metaron-uk/xbmc <xbmc@noreply.github.com>
Cc: Lars Op den Kamp <opdenkamp@gmail.com>
Onderwerp: Re: [xbmc] [PVR] API 2.2.0 Version Bump (c9b8057)

IMO, this should be a major API bump, as addons break, if their code will not be updated. @opdenkamp?

—
Reply to this email directly or view it on GitHub.


حاول تذكر في مكان ما إن الأرقام بنبدأها من صفر :)


@doudou Why do you change the buoyancy value? The flatfish model is sinking now. I think the flatfish must float if the thrusters are turned off.


*بعد السطر الخاص


Correct, our api is C

-------- Oorspronkelijk bericht --------
Van: Kai Sommerfeld <notifications@github.com>
Datum:27-07-2015  18:37  (GMT+01:00)
Aan: xbmc/xbmc <xbmc@noreply.github.com>
Cc: Lars Op den Kamp <opdenkamp@gmail.com>
Onderwerp: Re: [xbmc] [PVR] Add strRecordingId to EPG for API 2.2.0 (9b4876d)

As far as I know this is a C API, not C++, thus std::vector is a no go.

BTW: and please don't ask me why we have reference parameters in the signature of some API functions. The latter does in no way fit for plain C...

—
Reply to this email directly or view it on GitHub.


not necessary this return


The commit message indicates that error_message is used, but it is not :)


That could possibly work. The only problem with that is that won't work as expected on MSYS, since it does some [path conversion](http://www.mingw.org/wiki/Posix_path_conversion) (in order to work, you'd have to run `cmd //c start <url>` instead). I'm investigating to see if MSYS provides a utility for performing this conversion.


Also I'd take a look at your imports to make sure that you're not missing something, the working Polymer kit does this differently than you do: https://github.com/PolymerElements/polymer-starter-kit/blob/master/gulpfile.js#L12


Yes, you're right. I tuned it down because the buoyancy was way too high, but it seems that the system's mass got changed in the meantime. I'll remove it from here and figure out a better way to specify boyancy.


Slightly nitpicky, but I don't love the 'do_' stuff. If 'do_print_symbol' is just a more general version of 'print_symtab_symbol' then I think just 'print_symbol' is fine. But I don't feel strongly about it.


(for instance you do not appear to require gulp anywhere)


I'm surprised to see a flywheel.io directory in here. Seems you are using it name to package a scitran app inside a scitran repo. What am I missing?


Picking nits here, but there should be only one string literal here, i.e. don't mention `"$mono_libdir"` twice.

  const char *libdir_pattern = "$mono_libdir";


wait nevermind I'm just dumb and can't read. But still.


TeamCity xBIM :: XbimScripting :: Build [Build 27](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2142&buildTypeId=XBIM_XbimScripting_Build) is now running


TeamCity xBIM :: XbimScripting :: Build [Build 27](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2142&buildTypeId=XBIM_XbimScripting_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:38


s/request/requests; also, please document what "value" should be in both cases.


good


this is already being checked inside the function.


Use the `CMP_*` names.


:(
haithembelhaj why ?
I will be happy to help with the support


whoamiコマンド正しく処理されます。


Doubt it matters because we only call `f` we don't move or copy it.


s/overlay append/overlay-append/


**linting**


In that case I think the cleanest way to do it is to call the Windows API directly using `ShellExecuteW`.


rctDigis is correct under the existing naming scheme.  what is the ambiguity?  (the emulator is simRctDigis)


This is the line was changed


It would be awesome if this were `day`. Showing `1 d` next to `2 days` is bad.


@a-wortman ready for review again


Well, before this change we had:
```
system.build.installBootLoader = ... "PERL5LIB=${makePerlPath ... } ${pkgs.perl}/bin/perl ${./install-grub.pl}"
```


anshul test comment commit.


Thanks!


T


As mentioned in the earlier review: please either rename the whole function from `getxattr_malloc()` to `lgetxattr_malloc()` or add a bool `bool allow_symlinks` that allows choosing between `getxattr()` and `lgetxattr()` as internally used backend calls.

But right now it's really confusing since the function name is `getxattr_malloc()` though it internally doesn't use `getxattr()`, but `lgetxattr()`.


Test


Also, as mentioned in earlier review, I'd prefer if this would stay closer to glibc nomenclature, i.e. `fgetxattr_malloc()` rather than `getxattr_malloc_fd()`.


what i want to integrate the 'Home' button to make it Home. I try to did it to root_path, but the map does not appear unless I have to refresh the page.
Do you know how to fix it?


Looks good otherwise!


NetBeans projects only work as folders. Also, there is no such thing as a solution for NB.
If you have multiple projects, it's not possible to have them all in the same folder without using subfolders.
I guess it would be possible to make a switch for the case of a single project, but then your locations are all changed when you decide to add another project.


anshul commet.


This is the best.


Another test comment


can be merged


Ić pan w hój


Looks fine! I added comments to another note...

On Mon, Jul 27, 2015 at 1:04 PM, Brev Patterson <notifications@github.com>
wrote:

> Trying to merge in @cogmission's changes to a current version of the site.
> Looks like changes on PR # 219 were to an older version of the site?
> (I think! I'm not the best at git yet). :)
> This should be all the changes to `docs/` from
> `cogmission:htm-java-update-0.5.2` branch, going into a current version of
> repo.
>
> Please make sure this looks OK @cogmission @rhyolight thanks!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/brev/numenta.org/commit/3365d0eb980fd935241564794e01a7a8eab5b6b3
>

--
*With kind regards,*

David Ray
Java Solutions Architect

*Cortical.io <http://cortical.io/>*
Sponsor of:  HTM.java <https://github.com/numenta/htm.java>

d.ray@cortical.io
http://cortical.io


This causes "The 3rd Birthday" to crash on triangle button "change body power" effect. Tested revs from r770 to r790, last working is 789, so I recompiled PPSSPP without `fabsf` removal in getViewportX1/getViewportY1 and it works fine.


This should be `false`.


thanks, bill gates


I agree that *ideally* we should be using `std=c++14` instead of the former `std=c++1y` but frankly it seems like more trouble than it's worth to install a new compiler toolchain just for that.


what is the this content with the `that` ?


#PrintAllLinks(page)

Dependant on GetNextTarget.py

1 in, 0 out

Prints out all links on a webpage.


This should be generalized to "The packages needed to build the kernel source on your distribution". Debian-based distros have this as "build-essential", RH-based ones have "kernel-devel", etc. etc. Leave it more generalized.


Oh right which isn't a secondary script.

On Mon, Jul 27, 2015, 14:09 Eelco Dolstra <notifications@github.com> wrote:

> Well, before this change we had:
>
> system.build.installBootLoader = ... "PERL5LIB=${makePerlPath ... } ${pkgs.perl}/bin/perl ${./install-grub.pl}"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/fd5b273e82cf7e78a94479730cf1f51d3a19c637#commitcomment-12379042>
> .
>


add_fixtures_to_VS_solution (mbadams5) - x86_64-MacOS-10.9-clang: OK (1757 of 1757 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)


Looks like the copyright date could also use an update


I think we should just print the benchmark names, without any additional stuff, to not have to do pattern matching.  I'd prefer one benchmark per line, no commas.


This should not be removed.

Overall, I think we should clarify that this document is meant for 2015 kernel builds, and note that instructions for previous years builds are in the associated branches of this repo. This makes it easy to keep the documentation correct and updated (by pointing customers who want older builds to the correct information)


Shouldn't you be doing `||` instead of `&&`?


Again, this should stay removed


Oh, right, thanks. (1f6df56a2352b47f584fef1d7e900d2d47481af1)


this.props.hours.map(function (hour, index) {

->

```
this.props.hours.map((hour, index) => {
```

lets fat arrow this


sdafds


Good note, thanks


:rocket: :star2:


What's up with these shifts?


@skenet Can you [open a new issue](https://github.com/huytd/agar.io-clone/issues/new) on this? And can you also specify the OS and browser you are on?


Done in the next change


I've fixed our printing tests a7bfe5643b0a53aab2bc578d4b527a59f01cc280


hello


Shift right by 31 instead


hi


Good catch, this error has been in there for far too long.


getting an error. running on win 7 behind proxy, maltego carbon 3.5.3. licensed. pasting the error below. Python 2.7 installed.

Transform ThreatCrowd enrich IP returned with an error: ParseError at [row,col]:[7,56]
Message: Attribute name "error" associated with an element type "urlopen" must be followed by the ' = ' character.

Manual pull of URL and IP combo returned as expected. Was wondering if I need to upgrade to Chlorine version

I believe you have to make urllib2 proxy aware for use in corporate environments


Alas, the official [`Win32`](https://github.com/haskell/win32) library doesn't have it, but it shouldn't be too difficult to roll our own in this package. When I get a chance, I'll try to implement it.


nit: four space indents in this file.


I "always" run it from VS and put a breakpoint at the end of the test's `main()` function to see the output. There's some VS2013 extension for GoogleTest but I never got that working properly.


I agree with you. But I think that some video cards don't have support to VBOs. So, I think in some computers, this code will not work. But I am not sure. On the other hand, should we assume that if someone wants to execute the simulation, he has a good computer with a good video card?


this was making the list view reeaaaaallllyyy slow


Good note, and something that I should probably fix (the method that I use for getting the modules onto the target simply leave broken symlinks, but we should make this easier)


originalValues could be null, then this compare will always to be false.


Thanks for fixing this! I wasted hours last night on my Mega trying to figure out what I was doing wrong.. turns out my DmxMaster library was just out of date.


What? I'm gonna echo neobrain's comment from that other instruction here.


That is because the [GPX](http://www.topografix.com/GPX/1/1/#type_trksegType) format specifies a track as a list of track segments, and this was my idea of how to implement that easily.
>  A Track Segment holds a list of Track Points which are logically connected in order. To represent a single GPS track where GPS reception was lost, or the GPS receiver was turned off, start a new Track Segment for each continuous span of track data.


Unnecessary change, which also causes the lint error. It seems like you have not pull the current master code before applying your changes, so you will need to pull the master code in your branch first.


learning to love git


passing in global static config at this level seems anti react.. as compared to simply passing the bar height in as a prop


yeah, this probably isn't a great place for it. Not sure where is better though.


dada


this function to longer exists, right?


Address the comments marked [FIX]


Fixed


@igrigorik i gave that a shot, but ran into problems resetting state as clients will send the connection preface after an options upgrade for a post. see latest branch [commits](https://github.com/igrigorik/http-2/compare/master...kenichi:upgrade) and alternate `example/celluloid_server.rb` which handles upgrades.


The auto-generated bindings should already work for this function since it's not inline.


This code runs forever....


right! argh ;)


sdfsdfsadfa


@victordzrz ,You are right. I will change it to openrandom_get16b()%SLOTFRAME_LENGTH.

For the function, here I just want the function as simple as possible and yes, it didn't guarantee available offset can be found even they exists. The code here is temporary. It will be replaced by some other algorithm for cell selection in the future.


@AndyScherzinger , please, don't commit stuff without review. Open a PR even for documentation changes.

Anyway, this commit may stay.


As long as we're here we might consider reporting whether or not the peer is "insane"


Also, this way the Makefile in the NB project folder will not collide with a possible Makefile by gmake.


Ditto - the jsapi version of this should work..


the thought behind it was that it's the same format that is accepted by `--benchmarks`, but I'm also happy with printing one benchmark per line


Leave comments here's saying which parts correspond to the IF, ELSE, END.


最新的master分支，patch后mac上编译报错：
/Users/hsl4125/muduo/examples/protobuf/rpcbalancer/balancer_raw.cc:15:10: fatal error:
      'endian.h' file not found
#include <endian.h>
         ^
1 error generated.


10/10 would use EE3 now


*downloading now*


Also, if the else part is empty, you could check for that and avoid the extra jump.


It's also useful for `C-h k` I think.


Is this the reason why our kde tests fail?


ok nice!
thanks for that


hiaflksdjf;a;jo


I don't understand what you mean? What's the `definition-name`?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.830.04d3926c974d60b75b794e93c771c73ed00beed2](http://build.palaso.org/viewLog.html?buildId=60059&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:49


safsafdsf


just create a branch and start a pull request. Then automatically Travis will build with your changes @solael @vhuber


@jmwohl
The value for iconAnchor was off and caused the marker to drift along the map as you zoomed in or out. I opened up the icon image in GIMP to find the correct (I think...) anchor location.


You can load directly into `LOOPINC`.


Cfu in vs them g


The issue is created at https://openwsn.atlassian.net/browse/FW-437


TeamCity Subsonic :: Debug [Build 11](http://109.103.226.38:8512/viewLog.html?buildId=34601&buildTypeId=Subsonic_Debug) is now running


when running this on my local server i get this error:
Fatal error: Call to a member function find() on boolean in /Applications/XAMPP/xamppfiles/htdocs/engine/api/socialclub/index.php on line 37


Consider removing this return statement. I could see an argument for keeping it for clarity of control flow though.


TeamCity Subsonic :: Debug [Build 11](http://109.103.226.38:8512/viewLog.html?buildId=34601&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:24


TeamCity Subsonic :: Debug [Build 12](http://109.103.226.38:8512/viewLog.html?buildId=34603&buildTypeId=Subsonic_Debug) is now running


Arrays.equals() returns true if both arrays are null, isn't that what we want?


TeamCity Subsonic :: Debug [Build 12](http://109.103.226.38:8512/viewLog.html?buildId=34603&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:31


```ts
return links.resolvedJsxType = elemInstanceType;
```


fsadfdsfsda


818f7460f8567458fad688269e6b4cc220b2e4ad seems to be the issue not this commit, but that is a bit baffling. To be continued later.


مكن تشيل دي و تدخل بالكلام علي طول من غير "البرنامج الأول" :)


`updateWalker` appears to be an automatic way to fill the src attribute. Interesting! I really don't know why it isn't used.
In fact, `version` is not an attribute per se, but it makes the expression easier to update and read. It is better than write `name = mksh-51` and `mksh-R51` in the source URL. Maybe monitor thinks the same way, and then updates version accordingly.

Anyway, it is just something merely cosmetical - if it compiles, runs and doesn't fail, then there is no reason to a modification. It's more a curiousity, after all.
Let it be the way you wrote!


Why remove translated lines? are they in another sub file?


In order to preserve the specific variety of `any` that you have.


To je tak, kdyz se vyjadruju zavadejicim zpusobem jako politik :-) Promin.
Prosim odsud to `g.user...` uplne pryc, `inactive` nechat a `not g.user...` dat jako parametr misto `True` v `candidate_signup.html`. Tim ziskame univerzalnejsi funkcionalitu `inactive` v tomto makru.


:hankey:


Tested on SNI deployment. Resolved the issue perfectly. Cheers.


sdfsdafsd


I at least timed them in the mori-speech file. (It spans the whole 2x minutes of the ceremony)


thank you thank thank you thank you :)


Why do we need these three things?


SDFSAFSDFA


It exists only to support this deprecated tool.


SDFASFSDF


if min == max, range = max - min == 0; width = range / X == 0; num_terms = floorf(range / 0) + 1;


In a side-by-side comparison, here's what's changed:

1. New: `MOVE` always triggers first. Old: `MOVE` always triggered last.
2. New: `MOVE` never prevents `ENTER`. Old: `ENTER` may prevent `MOVE`

The implication is that `MOVE` is now triggered before `ENTER`, on the same cycle, which will cause some strange and unexpected behavior for handlers that expect to receive the `ENTER` event first.

Second, the `MOVE` event cannot be prevented by `ENTER` now. This one is probably not a big deal, since the next `MOVE` event will be triggered regardless.

Overall, I think due to point 1 above, this change will break expectations.


replace getWiki method, cache dowloaded markdown file


A bit of it, mostly about `@DataBoundSetter`; I include it here because it is especially relevant for Workflow compatibility: if you get it wrong, UI configuration from freestyle projects may still work, but _Snippet Generator_ and/or actual runtime behavior from Groovy may be broken.


GHFGDF


Why do we need these things?


> Correct, our api is C
@opdenkamp How fit reference parameters into this story? We're having > 10 functions in the API using reference parameters. Example: https://github.com/xbmc/xbmc/blob/master/xbmc/addons/include/xbmc_pvr_dll.h#L273


Iteration order over this map affects test data presentation. It's better to use lists of pairs here. Not only that, but it actually is intended to be a list of pairs, and not a mapping from function to it's their arity.

Also, there's no need for it to be protected. Avoid exposing state when it's not necessary.


A newline.


CNAME 도 build 로 복사되게 해줘야 겠네염


I like that. Starting to look like Javascript :laughing:


Again, it's a list of declarations, not a mapping.


Но здесь configFile еще не создан, тогда надо открывать из метода создания этого файла.


Blank lines.


Действительно, лучше возвращать строчку, так меньше будет стримов, кажется.


First Commit


This looks like it may be a mistake.  The name of the unit should be `deis-mesos-zk` without the `@1`.

This seems to have broken `deisctl refresh-units`-- it throws a 404 on the final download.


Use utility methods in Pair for instantiation.


I did not intend for this to be committed, and we've been trying to resolve it in IRC.


Don't think age matters, but I'd prefer to keep daySuffixes off (i.e. 12 instead of 12th)


Geen probleem, graag gedaan. :)


問題なく動きました。
できれば前のnameコマンドの際”名前の変更を全員に知らせる”部分などがありましたが、postメソッドに変えてもらえるといいと思います。


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


Thanks for contributing & improving the code. I'm going to fold these changes back in.

I've done some experiments on large datasets and found that converting Matrix - > matrix isn't necessary and has too much memory overhead. I plan on updating that accordingly as well as include a fully parallel stars method that uses a queuing system, based on the BatchJobs package.


Se tivesse alguma lógica neste commit, ele seria desconsiderado.


TeamCity Subsonic :: Debug [Build 13](http://109.103.226.38:8512/viewLog.html?buildId=34605&buildTypeId=Subsonic_Debug) is now running


Ok!


TeamCity Subsonic :: Debug [Build 13](http://109.103.226.38:8512/viewLog.html?buildId=34605&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:28